Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Problem with translation system #91

Closed
nvdaes opened this issue May 11, 2024 · 2 comments
Closed

Problem with translation system #91

nvdaes opened this issue May 11, 2024 · 2 comments

Comments

@nvdaes
Copy link
Contributor

nvdaes commented May 11, 2024

@josephsl , the l10n server has notified the following:

Running addon2svn for wintenApps
Already on 'stable'
Your branch is up to date with 'origin/stable'.
From github.com:nvdaaddons/wintenApps
   31ad7946..ad3337d  stable     -> origin/stable
HEAD is now at ad3337d Add-on template: incorporate 2024.2 version follow-up such as custom braille translation tables (Windows App Essentials does not include them)
scons: Reading SConscript files ...
  File "/home/nvdal10n/mr/addons/wintenApps/sconstruct", line 193

                manifest += f"{'\n'.join(manifest_brailleTables)}\n"

                                                                    ^

SyntaxError: f-string expression part cannot include a backslash

mr addon2svn: command failed

So, translators are not receiving new strings for this add-on.
Cheers

@josephsl
Copy link
Owner

josephsl commented May 11, 2024 via email

@josephsl
Copy link
Owner

Hi,

Traced to F string expression being invalid if backslash characters are included in the expression portion. This issue is resolved in Python 3.12, and a fix is provided in latest add-on template (commit b82cbfc).

Thanks for the initial report.

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

No branches or pull requests

2 participants