Project

General

Profile

Développement #55822

Retirer les numéros de lignes des fichiers de traduction

Added by Valentin Deniaud over 3 years ago. Updated over 3 years ago.

Status:
Fermé
Priority:
Normal
Category:
-
Target version:
-
Start date:
26 July 2021
Due date:
% Done:

0%

Estimated time:
Patch proposed:
Yes
Planning:
No

Description

En passant --add-location=file, ce qui devrait permettre d'avoir des diffs lisibles.

Morceaux choisis du salon où l'idée a été validée démocratiquement,

fpeters: cela étant les builds de .deb se font encore avec django 1.11, si ça n'existait pas, c'est raté.
bdauvergne: ça n'existait pas
bdauvergne: mais le build c'est juste compile_messages
fpeters: en effet, très bien.

bdauvergne: "tox -e manage -- makemessages --add-location file" dans update-locate.sh, ou alors une cible dans tox et on vire le script

Files

Associated revisions

Revision 6ed57791 (diff)
Added by Valentin Deniaud over 3 years ago

commands: add custom makemessages command (#55822)

Its purpose is to stop including line numbers in .po translation files.

History

#1

Updated by Valentin Deniaud over 3 years ago

bdauvergne: tox -e manage -- makemessages --add-location file dans update-locate.sh, ou alors une cible dans tox et on vire le script

Bon en vrai j'ai rien compris à l'utilité de ce script qui n'a l'air appelé nulle part, et je ne vois pas ce que tox vient faire dans l'histoire. Je me contenterais bien de faire dans ce fichier

-./authentic2-ctl makemessages
+./authentic2-ctl makemessages --add-location file

#2

Updated by Benjamin Dauvergne over 3 years ago

Valentin Deniaud a écrit :

bdauvergne: tox -e manage -- makemessages --add-location file dans update-locate.sh, ou alors une cible dans tox et on vire le script

Bon en vrai j'ai rien compris à l'utilité de ce script qui n'a l'air appelé nulle part, et je ne vois pas ce que tox vient faire dans l'histoire. Je me contenterais bien de faire dans ce fichier
[...]

tox c'est pour éviter d'avoir à construire un venv, le script fait pareil en plus moche.

#3

Updated by Valentin Deniaud over 3 years ago

Benjamin Dauvergne a écrit :

tox c'est pour éviter d'avoir à construire un venv, le script fait pareil en plus moche.

Moi la dépendance à tox je trouve ça moche, et ça ne me dit pas à quoi sert le script. Si c'est important ça sera fait dans un autre ticket, pour l'instant celui-ci sert surtout à annoncer le changement aux trads. Patch minimaliste, donc.

#4

Updated by Valentin Deniaud over 3 years ago

Approche alternative sans toucher au script, pour faire comme dans wcs (#55848)

#5

Updated by Benjamin Dauvergne over 3 years ago

  • Status changed from Solution proposée to Solution validée
#6

Updated by Valentin Deniaud over 3 years ago

  • Status changed from Solution validée to Résolu (à déployer)
commit 6ed57791acaa30047bec4c08c6aeb71a5bcf2a24
Author: Valentin Deniaud <vdeniaud@entrouvert.com>
Date:   Tue Jul 27 10:09:09 2021 +0200

    commands: add custom makemessages command (#55822)

    Its purpose is to stop including line numbers in .po translation files.
#7

Updated by Frédéric Péters over 3 years ago

  • Status changed from Résolu (à déployer) to Solution déployée

Also available in: Atom PDF