Skip to content

Latest espeak to resolve italian pronunciation#7115

Closed
feerrenrut wants to merge 8 commits into
masterfrom
latestEspeakToResolveItalianPronunciation
Closed

Latest espeak to resolve italian pronunciation#7115
feerrenrut wants to merge 8 commits into
masterfrom
latestEspeakToResolveItalianPronunciation

Conversation

@feerrenrut

Copy link
Copy Markdown
Contributor

Update Espeak to commit 5226ff0

Some changes had to be made to build system so support this Espeak update.
We must now compile and link and include in the search paths ucd-tools. To do this, I diffed the the visual studio project files in the espeak project.

I have excluded 'zhy` dictionary since it was failing to build.

A try build has been given to the user asking for better support, we have not yet had feedback.

This is to see if it resolves issues with pronunciation in Italian
zhy has been excluded as it wont compile.
feerrenrut added a commit that referenced this pull request May 3, 2017
Merge branch 'latestEspeakToResolveItalianPronunciation' into next
@jcsteh

jcsteh commented May 8, 2017

Copy link
Copy Markdown
Contributor

This update introduces some ugly regressions. According to espeak-ng/espeak-ng#250, at least some of these should be fixed in the latest espeak-ng commit. However, @derekriemer noted on Gitter that he can't get this commit to build with NVDA:

ctype.obj : error LNK2019: unresolved external symbol _ucd_properties referenced
in function _ucd_isalnum
readclause.obj : error LNK2019: unresolved external symbol _clause_type_from_cod
epoint referenced in function _ReadClause
build\x86\espeak\espeak.dll : fatal error LNK1120: 2 unresolved externals

@derekriemer also did a bisect and discovered that this occurs in the very first espeak-ng commit after the one we now use in next.

@feerrenrut, can you take a look at updating this PR to the latest espeak-ng commit? Thanks.

@feerrenrut feerrenrut requested a review from michaelDCurran May 9, 2017 04:10
@Brian1Gaff

Brian1Gaff commented May 9, 2017 via email

Copy link
Copy Markdown

feerrenrut added a commit that referenced this pull request May 11, 2017
re-incubate #7115 after updating espeak to a newer commit.
Merge branch 'latestEspeakToResolveItalianPronunciation' into next
feerrenrut added a commit that referenced this pull request May 16, 2017
Merge branch 'latestEspeakToResolveItalianPronunciation' into next
Fixes part of #7167
selecting Norwegian Bokmål no longer causes an error.
@feerrenrut feerrenrut requested a review from jcsteh May 26, 2017 04:20

@jcsteh jcsteh 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.

Looks good!

Comment thread source/synthDrivers/espeak.py Outdated
voices=OrderedDict()
for v in _espeak.getVoiceList():
l=v.languages[1:]
# #7167: Some languages names require unicode characters EG: Norwegian Bokmål

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.

nit: I think this would be more readable as "Some language names contain Unicode characters".

Clarified comment
feerrenrut added a commit that referenced this pull request May 26, 2017
Merge branch 'latestEspeakToResolveItalianPronunciation' into next
To commit c5afdbe0d1292a6f1d95e3794b62e4be93d7a6cd
feerrenrut added a commit that referenced this pull request May 26, 2017
This is done as a separate merge to next, in case there are any major
issues. Please roll back to the last 'next' commit.
Merge branch 'latestEspeakToResolveItalianPronunciation' into next
feerrenrut added a commit that referenced this pull request Jun 13, 2017
This commit will revert 8 updates to espeak that have been incubated to the 'next'
branch, see #7115

Revert "Update espeak-ng"
commit 362af4a.

Revert "Review actions for #7115"
commit b81d541.

Revert "Allow unicode characters in espeak language names"
commit b2b1ca5.

Revert "Update espeak to fix #7167"
commit 8fa4335.

Revert "Fix build errors with latest espeak."
commit 6e005f8.

Revert "Update to the latest espeak"
commit 36a381d.

Revert "Get espeak building again"
commit a184415.

Revert "Update espeak-ng"
commit 22e2931.
@feerrenrut

Copy link
Copy Markdown
Contributor Author

The changes for this PR are going to be reverted from 'next'. We will first incubate the most recent release of espeak-ng 1.49.1. Once that is incubated and (if considered stable) merged to master, we will resume updating next on a regular basis. For now I will close this PR.

feerrenrut added a commit that referenced this pull request Aug 1, 2017
Clarified comment
@feerrenrut feerrenrut deleted the latestEspeakToResolveItalianPronunciation branch September 21, 2017 01:30
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.

5 participants