Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Report Windows Server 2025 in php_uname #16838

Closed
wants to merge 1 commit into from

Conversation

Quppa
Copy link
Contributor

@Quppa Quppa commented Nov 17, 2024

Windows Server 2025 was released on 2024-11-01 with a major build number of 26100 (matching Windows 11 24H2). With this patch php_uname will return the correct version string (currently Windows Server 2022).

Previous related PR for Windows Server 2022 and earlier: #7816

I'm targeting master but please let me know if I should choose PHP-8.4 or something else as the base branch.

Copy link
Member

@cmb69 cmb69 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Good catch. Thank you!

I think this should be applied to PHP-8.4 (no need to change the base branch on the PR, though), if @php/release-managers-84 have no objections.

@NattyNarwhal
Copy link
Member

NattyNarwhal commented Nov 17, 2024

I think due to the requirement that GA = last RC, this might need to wait until .1. (edit: But it does seem the rule is not really binding per this thread, and this seems cosmetic, so if that's the case, I don't have a problem with merging this if other RMs are OK with it.)

@cmb69 cmb69 closed this in f086eaa Nov 26, 2024
@cmb69
Copy link
Member

cmb69 commented Nov 26, 2024

Thank you! Applied to PHP-8.4 and merged up.

@Quppa Quppa deleted the windows-server-2025 branch November 27, 2024 14:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants