Skip to content

Update espeak-ng to resolve espeak-ng/espeak-ng#1031#18633

Merged
seanbudd merged 1 commit into
nvaccess:masterfrom
codeofdusk:update-espeak-20250806
Aug 6, 2025
Merged

Update espeak-ng to resolve espeak-ng/espeak-ng#1031#18633
seanbudd merged 1 commit into
nvaccess:masterfrom
codeofdusk:update-espeak-20250806

Conversation

@codeofdusk

Copy link
Copy Markdown
Contributor

Link to issue number:

espeak-ng/espeak-ng#1031.

Summary of the issue:

Espeak puts adjectival stress on "has been" in English in all cases, even when it's used as a verb. This was resolved upstream, and NVDA should take in this fix.

Description of how this pull request fixes the issue:

Updates espeak-ng to commit 3b8ef3d.

Testing strategy:

Verified that "has been" is pronounced correctly as a verb form (en-gb, max variant).

Known issues with pull request:

None known.

Code Review Checklist:

  • 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 August 6, 2025 19:19
@codeofdusk codeofdusk requested a review from a team as a code owner August 6, 2025 19:19
@codeofdusk codeofdusk requested a review from seanbudd August 6, 2025 19:19

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 eSpeak NG dependency to resolve a pronunciation issue where "has been" was incorrectly stressed as an adjective instead of a verb form in English.

  • Updated eSpeak NG submodule from commit a4ca101 to 3b8ef3d
  • Updated documentation to reflect the new commit hash and issue reference
  • Incorporates upstream fix for English pronunciation stress patterns

Reviewed Changes

Copilot reviewed 3 out of 3 changed files in this pull request and generated 1 comment.

File Description
user_docs/en/changes.md Updated changelog entry with new eSpeak NG commit hash and added issue reference
projectDocs/dev/createDevEnvironment.md Updated development documentation with new eSpeak NG commit requirement
include/espeak Updated git submodule pointer to new eSpeak NG commit

Comment thread user_docs/en/changes.md
@seanbudd seanbudd merged commit 41f862d into nvaccess:master Aug 6, 2025
22 checks passed
@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