search

nvaccess / mrconfig

'myrepos' configuration files for nvda translations and addons
GNU General Public License v3.0
5 stars 14 forks source link

[Register add-on in translation system]: MathCAT #110

Closed nvdaes closed 1 year ago

nvdaes commented 1 year ago

Add-on name

MathCAT

Add-on maintainer

@NSoiffer

Stable branch

github-actions[bot] commented 1 year ago

The pot files were built from the stable branch.

seanbudd commented 1 year ago

The add-on has been registered, but there are some encoding issues with MathCAT, cc @nsoiffer

scons: Reading SConscript files ...
scons: done reading SConscript files.
scons: Building targets ...
Generating pot file MathCAT.pot
xgettext: Non-ASCII comment at or before addon/globalPlugins/MathCAT/MathCAT.py:8.
          Please specify the source encoding through --from-code or through a comment
          as specified in http://www.python.org/peps/pep-0263.html.

scons: *** [MathCAT.pot] Error 1
scons: building terminated because of errors.
mr addon2svn: command failed

mr addon2svn: finished (1 failed)
nvdaes commented 1 year ago

The mentioned problem should be fixed in https://github.com/NSoiffer/MathCATForPython/commit/fa458cd0029d6da16088e32ca11e4d1691331493

I've been invited by Neil as a collaborator to his repo for tasks like this one. Please let us know if the problem persists.

nvdaes commented 1 year ago

@seanbudd , seems that some kind of problem persists since translations haven't been merged for this add-on. Can you check this please? Thanks.

seanbudd commented 1 year ago

There's been another issue:

mr addon2svn: /home/nvdal10n/mr/addons/MathCAT
Running addon2svn for MathCAT
Already on 'stable'
Your branch is up to date with 'origin/stable'.
HEAD is now at fa458cd Add coding declaration
scons: Reading SConscript files ...
scons: done reading SConscript files.
scons: Building targets ...
Generating pot file MathCAT.pot
xgettext: Non-ASCII string at addon/globalPlugins/MathCAT/MathCATPreferences.py:74.
          Please specify the source encoding through --from-code or through a comment
          as specified in http://www.python.org/peps/pep-0263.html.

scons: *** [MathCAT.pot] Error 1
scons: building terminated because of errors.
mr addon2svn: command failed
NSoiffer commented 1 year ago

I pushed the fix. I hope to build a new MathCAT later tonight.

nvdaes commented 1 year ago

Thanks Sean: Neil has fixed the coding declaration and I've merged the code to the stable branch, so translations should be merged properly next Friday.

El 14/08/2023 a las 5:09, Sean Budd escribió:

There's been another issue:

mr addon2svn: /home/nvdal10n/mr/addons/MathCAT
Running addon2svn for MathCAT
Already on 'stable'
Your branch is up to date with 'origin/stable'.
HEAD is now at fa458cd Add coding declaration
scons: Reading SConscript files ...
scons: done reading SConscript files.
scons: Building targets ...
Generating pot file MathCAT.pot
xgettext: Non-ASCII string at addon/globalPlugins/MathCAT/MathCATPreferences.py:74.
           Please specify the source encoding through --from-code or through a comment
           as specified in http://www.python.org/peps/pep-0263.html.

scons: *** [MathCAT.pot] Error 1
scons: building terminated because of errors.
mr addon2svn: command failed
seanbudd commented 1 year ago

I've confirmed that translations appear to be working now