Skip to content

Update Espeak to Release 1.50#10572

Merged
feerrenrut merged 2 commits into
masterfrom
UpdateEspeak
Dec 4, 2019
Merged

Update Espeak to Release 1.50#10572
feerrenrut merged 2 commits into
masterfrom
UpdateEspeak

Conversation

@feerrenrut

Copy link
Copy Markdown
Contributor

Link to issue number:

None

Summary of the issue:

espeak-ng/espeak-ng has a new release.

Description of how this pull request fixes the issue:

Update to version 1.50 of espeak-ng

Testing performed:

Built NVDA locally, ran with espeak as the configured synthesizer, tested several voice variants and languages.

Known issues with pull request:

Many lines were printed to my console window:

Bad voice attribute: fast_test
Bad voice attribute: language

Change log entry:

Changes:

eSpeak-ng updated to version 1.50

@LeonarddeR

Copy link
Copy Markdown
Collaborator

Many lines were printed to my console window:

Was this also the case with master?

@feerrenrut

Copy link
Copy Markdown
Contributor Author

Was this also the case with master?

No.

@zstanecic

Copy link
Copy Markdown
Contributor

I assume that we need to check their voice variant files and edit them accordingly.

@zstanecic

Copy link
Copy Markdown
Contributor

I mean, we need to edit our language variants files, for example, kaukovalta or fast

@Brian1Gaff

Brian1Gaff commented Dec 4, 2019 via email

Copy link
Copy Markdown

@feerrenrut

Copy link
Copy Markdown
Contributor Author

This message comes from /src/libespeak-ng/voices.c line 873. It doesn't look like anything in the area has changed recently.

I didn't notice any issues with the voices. I think I will merge this to get wider testing.

@feerrenrut feerrenrut merged commit 99869bb into master Dec 4, 2019
@nvaccessAuto nvaccessAuto added this to the 2019.3 milestone Dec 4, 2019
@feerrenrut feerrenrut mentioned this pull request Dec 5, 2019
@lukaszgo1

Copy link
Copy Markdown
Contributor

@feerrenrut In #10581 you've created branch with the same name as the one for this PR differing only in the capitalization of u. As this causes new branch to appear with each pull can one of these be deleted?

@feerrenrut feerrenrut deleted the UpdateEspeak branch December 6, 2019 09:15
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.

6 participants