Windows versions: recognize build 22621 as Windows 11 22H2#13867
Conversation
| * Then in the Installation details tree view, under Desktop for C++, Optional, ensure the following are selected: | ||
| * MSVC v142 - VS 2019 C++ x64/x86 build tools | ||
| * Windows 11 SDK (10.0.22000.0) | ||
| * Windows 11 SDK (10.0.22621.0) |
There was a problem hiding this comment.
It might be wise to wait for this version of SDK being available on AppVeyor before asking users to use it locally when building, just to make sure there are no unexpected regressions.
|
Hi, I expect that will be the case this fall. I can revert the readme commit just before releasing this PR from draft if NV Access agrees with this comment. Thanks.
|
|
Hi, A slight complication arose today: build 22622 is now flighting to beta channel Insiders, identical to 22621 apart from features being enabled. I plan not to add 22622 unless Microsoft releases a feature update with this build number in the future. Thanks. |
|
Hi, If I'm reading Appveyor installed software list correctly, Windows 11 SDK 10.0.22621 can be used in Appveyor, so no revert necessary. The only thing remaining is release of Windows 10 build 19045 to release preview Insiders, and once that happens, I'll remove the draft flag unless @codeofdusk says detecting SV2 (build 22621/nickel) is needed for enhanced Windows Console support. Thanks. |
|
Hi, It appears build 22621 SDK is available in Visual Studio 2022 appveyor image. Also, even with signs poiting to Windows 10 Version 22H2 coming to release preview Insiders in August (at the earliest), I think it would be best to at least let NVDA work with Nickel so Bill and others can work on proper SV2 support. Therefore, I will:
And then will remove draft PR flag so it can be ready for 2022.4. Thanks. |
|
Hi, Follow-up: as for 10.0.19045 recognition, I'll send a follow-up PR for it once that build comes to Windows Insiders. Thanks. |
|
Are we 100% sure that 22621 is SV2 final? It is currently in the external release preview ring but @carlos-zamora seemed to think that 22622 might be the shipped build. CC @DHowett perhaps for additional clarity? |
|
Hi, VS2019 image uses 22000 whereas VS2022 image has upgraded to 22621. Thought the SDK notice was reverted but perhaps not (the latest commit from me was a few minutes ago). Thanks.
|
|
My comment was in regards to Windows version detection. |
|
Hi, I’m assuming 22621 as that’s what the SDK says unless the shipping version will indeed be 22622 (beta), and if folks from Terminal team says so, I can either replace WIN11_22H2 entry or add 22622 to winVersion module. I advise holding off on further reviews for this PR until we get a word from Microsoft people as this PR can confuse everyone (not only NV Access and Microsoft, but also folks working on other products including various screen readers and apps). For NV Access folks, build 22622 is an enablement package on top of 22621 with features being tested by beta channel Insiders and UBR (update build revision, the fourth number in major.minor.build.revision) is higher than what release preview Insiders are running. Thanks.
|
See test results for failed build of commit 426d8660fa |
|
Hi, Major update: build 19045 (Windows 10 Version 22H2) was released to release preview Insiders on July 28, 2022. Therefore a PR to add 19045 will be posted soon, and I think that one can be merged first before this PR (that PR will target 2022.4). Thanks. |
) Partially closes #13845 Windows 10 part of #13867 Summary of the issue: Recognize Windows 10 build 19045 as Version 22H2. Description of user facing changes None Description of development approach Tested through Windows Insider Program - added Windows 10 build 19045 metadata. Testing strategy: Manual testing: instal Windows 10 build 19045 release preview build to make sure it is recognized as 22H2 by NVDA.
|
Blocking #13297. |
See test results for failed build of commit e777ac75d8 |
|
Given that we haven't heard from @DHowett or @carlos-zamora on the Windows release strategy and I'd really like to see #13297 unblocked to allow for wide testing early in the 2022.4 release cycle, I think it's reasonable to set 22621 as the SV2 build number for now. As 22621 is safely in the SV2 build range and we usually check for "at least" a given build in NVDA tests, even if SV2 final is 22622, NVDA shouldn't be affected. Thoughts @josephsl? |
|
Hi, I’ve come to the same conclusion recently, so I’ll take care of that tonight (just need to remove SConstruct commit). Thanks for the follow-up.
|
See test results for failed build of commit 78bc83839d |
|
Hey all! Sorry for the radio silence. 22621 is the final version number for SV2. It is my understanding that the update being tested in the Beta channel today will not increment the version number (which means that Beta will stay a higher version number than RTM.) |
|
Thanks @DHowett for clarifying! |
See test results for failed build of commit 78bc83839d |
See test results for failed build of commit 911e87fe9f |
|
Should this have the blocked label removed? |
Link to issue number:
Closes parts of #13845
Summary of the issue:
Recognize Windows 22H2 releases. In part 1 (this pull request), recognize 10.0.22621 as Windows 11 Version 22H2 (Nickel/Sun Valley 2).
Description of user facing changes
No user facing changes
Description of development approach
Verified that the following builds will be tied to upcoming Windows feature updates:
Build 22621 is available to release preview Windows Insiders, build 19045 forthcoming.
Testing strategy:
Manual testing (ideally on virtual machines):
Control+NVDA+Z) and type "import winVersion; winVersion.getWinVer()" without quotes. Verify that version returned is "Windows 11 22H2".Known issues with pull request:
Although it is highly unlikely, Windows 10 22H2's build number could change in the future. The sure sign is which build will be released to Windows Insiders, and indication is that it will be 19045.
Change log entries:
None
Code Review Checklist:
Additional context:
Originally this PR added winVersion entry for Windows 10 Version 22H2 but is deferred to a later PR.