Skip to content

Conversation

@SaschaCowley
Copy link
Member

No description provided.

josephsl and others added 3 commits August 7, 2025 09:51
…sed via Widows Registry (#18621)

Closes #18617 

Summary of the issue:
In Windows 8.1, attempting to access Windows Registry to obtain update
build revision (UBR) results in file not found error and the snapshot
installer no longer starts.

Description of user facing changes:
Alpha/beta builds can be installed on Windows 8.1 again.

Description of developer facing changes:
If UBR cannot be obtained, build revision will be set to 0.

Description of development approach:
in winVersion module, put build revision assignment inside a try block,
and set build revision to 0 if an exception occurs while accessing the
registry. In Windows 8.1, this will result in NVDA reporting
"6.3.9600.0" which is acceptable as Windows 8.1 (client version) is no
longer supported by Microsoft (Windows Server 2012 R2 is supported under
extended security updates program).

Testing strategy:
Tested in Windows 8.1 virtual machine:
* nvda_snapshot_beta-51631,a18ebb3 cannot be started
* nvda_snapshot_try-win81UBRRegKey-37682,e6d54c46 works normally
Thanks to @wmhn1872265132 for testing

Known issues with pull request:
None
Amharic is a language with it's own alphabet, numerals, punctuation
marks and other symbols. This list contains punctuation marks.
This pull request updates translations to languages being tracked from
Crowdin.

Co-authored-by: GitHub Actions <>
Copilot AI review requested due to automatic review settings August 7, 2025 00:27
@SaschaCowley SaschaCowley requested a review from a team as a code owner August 7, 2025 00:27
@SaschaCowley SaschaCowley requested a review from seanbudd August 7, 2025 00:27
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Copilot encountered an error and was unable to review this pull request. You can try again by re-requesting a review.

@SaschaCowley SaschaCowley merged commit 54c265f into master Aug 7, 2025
7 of 8 checks passed
@github-actions github-actions bot requested a deployment to snapshot August 7, 2025 00:59 Abandoned
@github-actions github-actions bot requested a deployment to snapshot August 7, 2025 01:55 Abandoned
@SaschaCowley SaschaCowley added this to the 2025.3 milestone Aug 18, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants