Transifex Client now supports Python 3.5 in addition to Python 2.7:
https://ci.appveyor.com/project/transifex/transifex-client/branch/master
The script './pullTranslations.py' is now compatible with Python 3 as well as Python 2.
Details
Details
- Reviewers
• leper vladislavbelov Itms
Not applicable.
Diff Detail
Diff Detail
- Lint
Lint Skipped - Unit
Unit Tests Skipped
Event Timeline
| source/tools/i18n/pullTranslations.py | ||
|---|---|---|
| 33 | If you move, you need to exit if it's incompatible version, like: if sys.version_info[0] != 3: | |
| source/tools/i18n/maintenanceTasks.sh | ||
|---|---|---|
| 36 | If we move one of these, why not the other too? Also these blank lines help with splitting things into sections, I'd keep them. | |
| source/tools/i18n/pullTranslations.py | ||
| 49 | Why not just hide this behind a version check? Unless we are moving to python 3 exclusively. | |
| ./source/tools/i18n/maintenanceTasks.sh | ||
|---|---|---|
| 36 ↗ | (On Diff #1984) | Why 2 was removed only from a line below? Why not here? |
| ./source/tools/i18n/maintenanceTasks.sh | ||
|---|---|---|
| 36 ↗ | (On Diff #1984) | Because it's a Python 2 script that I didn't check for compatibility. If you care, do it yourself -- I have other differentials upcoming that are more important. |
| ./source/tools/i18n/maintenanceTasks.sh | ||
|---|---|---|
| 36 ↗ | (On Diff #1984) | Then it's not good way to port, because now you need to have 2 versions of Python instead of 1, so I don't see profits in this case. Scripts work with python2 too. |
| ./source/tools/i18n/maintenanceTasks.sh | ||
|---|---|---|
| 36 ↗ | (On Diff #1984) | Porting all of 0.A.D. to Python 3 isn't something I can attempt (yet), my knowledge is too poor for even an effort estimate. But at least now the script I patched works for both Python 3 and Python 2 now, so that's not a concern for this differential any more. |
Comment Actions
Might be ok without the shell script changes.
Handing this to Itms since he's IIRC responsible for the translation update box.
| ./source/tools/i18n/maintenanceTasks.sh | ||
|---|---|---|
| 36 ↗ | (On Diff #1984) | updateTemplates depends on pology which still is python2, though there is some outdated branch lifting that to python3 https://github.com/KDE/pology/tree/python3. Could try using that upstream, but likely another patch |
| 41 ↗ | (On Diff #1984) | Not sure if we should support old (which are being nuked everywhere) or newer (not existing yet) versions. Maybe forcing python3 is good enough. |