Skip to content

Use later liblouis commit#18995

Merged
seanbudd merged 1 commit into
nvaccess:masterfrom
LeonarddeR:fixLiblouis
Sep 29, 2025
Merged

Use later liblouis commit#18995
seanbudd merged 1 commit into
nvaccess:masterfrom
LeonarddeR:fixLiblouis

Conversation

@LeonarddeR

Copy link
Copy Markdown
Collaborator

Link to issue number:

Fixup for #18848

Summary of the issue:

The v3.35.0 tag on liblouis git repository is some commits behind the actual 3.35 release. The tagged version we used before didn't have its version number changed, so NVDA reports it as 3.34.0

Description of user facing changes:

NOne

Description of developer facing changes:

None

Description of development approach:

Update submodule to the most recent commit from september 1st.

Testing strategy:

Ensure that version is properly reported in log

Known issues with pull request:

None

Code Review Checklist:

x] Documentation:

  • Change log entry
  • User Documentation
  • Developer / Technical Documentation
  • Context sensitive help for GUI changes
  • Testing:
    • Unit tests
    • System (end to end) tests
    • Manual testing
  • UX of all users considered:
    • Speech
    • Braille
    • Low Vision
    • Different web browsers
    • Localization in other languages / culture than English
  • API is compatible with existing add-ons.
  • Security precautions taken.

Copilot AI review requested due to automatic review settings September 25, 2025 11:32
@LeonarddeR LeonarddeR requested a review from a team as a code owner September 25, 2025 11:32
@LeonarddeR LeonarddeR requested a review from seanbudd September 25, 2025 11:32

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull Request Overview

Updates the liblouis submodule to use a more recent commit from September 1st to fix version reporting issues. The v3.35.0 tag was outdated and caused NVDA to incorrectly report the liblouis version as 3.34.0 instead of 3.35.0.

  • Updates liblouis submodule commit hash to a more recent version
  • Fixes version number reporting in NVDA logs

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

@seanbudd seanbudd merged commit c0a7bb9 into nvaccess:master Sep 29, 2025
38 of 40 checks passed
@seanbudd

Copy link
Copy Markdown
Member

Thanks @LeonarddeR

@github-actions github-actions Bot added this to the 2026.1 milestone Sep 29, 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.

3 participants