library/shutil.po: quelques fuzzies #184

deronnax · 2023-08-22T08:32:04Z

deronnax commented

2023-08-22 08:32:04 +00:00

No description provided.

deronnax added 1 commit 2023-08-22 08:32:06 +00:00

ci/woodpecker/pr/woodpecker Pipeline failed Details

884ad35d4a

library/shutil.po: quelques fuzzies

ChristopheNan approved these changes 2023-08-23 12:02:44 +00:00

library/shutil.po

						
				@ -889,3 +888,1 @@

				msgstr ""

				#: library/shutil.rst:658

				#: library/shutil.rst:649 library/shutil.rst:658

ChristopheNan commented

2023-08-23 12:02:37 +00:00

Tu es sûr de ton coup ? En particulier pour les prochains make merge ?

deronnax commented

2023-08-24 08:28:34 +00:00

Hum non, je ne sais pas comment c'est arrivé là

deronnax commented

2023-08-27 10:11:46 +00:00

Je ne vois pas le diff comme ça dans mon terminal, il est normal, on dirait une erreur de Gitea.

ChristopheNan commented

2023-08-27 22:40:11 +00:00

Lorsque je récupère ta PR, j'ai bien ce qui est affiché ici. Poedit ne dit rien.

Lorsque j'ouvre le même fichier en provenance de upstream/3.11 avec mon Poedit, il m'affiche une fenêtre d'erreur avec le contenu suivant (powrap semble avoir du mal aussi avec ce fichier en provenance de upstream/3.11) :

Poedit a automatiquement corrigé le contenu non valide du fichier « shutil.po ».

Le fichier contenait des éléments dupliqués, ce qui n'est pas autorisé dans les fichiers PO et empêcherait le fichier d'être utilisé. Poedit a résolu le problème, mais vous devriez examiner les traductions de tous les éléments marqués comme nécessitant une révision et les corriger si nécessaire.

Lorsque je récupère ta PR, j'ai bien ce qui est affiché ici. Poedit ne dit rien. Lorsque j'ouvre le même fichier en provenance de upstream/3.11 avec mon Poedit, il m'affiche une fenêtre d'erreur avec le contenu suivant (powrap semble avoir du mal aussi avec ce fichier en provenance de upstream/3.11) : **Poedit a automatiquement corrigé le contenu non valide du fichier « shutil.po ».** Le fichier contenait des éléments dupliqués, ce qui n'est pas autorisé dans les fichiers PO et empêcherait le fichier d'être utilisé. Poedit a résolu le problème, mais vous devriez examiner les traductions de tous les éléments marqués comme nécessitant une révision et les corriger si nécessaire.

deronnax commented

2023-08-29 09:16:19 +00:00

haaan, c'est pété upstream, OK. Oui on va faire une passe. Bien vu, merci Christophe.

ChristopheNan commented

2023-09-23 18:25:40 +00:00

J'essaie de reproduire le « cassé » :

[chris@bat tmp]$ git clone https://git.afpy.org/AFPy/python-docs-fr.git d && cd d
[chris@bat d]$ grep '^CPYTHON_CURRENT_COMMIT' Makefile  #  future destination
CPYTHON_CURRENT_COMMIT := afa24d52b821c2020e0966f96a6205bca7db85e9
[chris@bat d (3.11)]$ git log | head -n 40 # trouve une rev avant cassage
...
[chris@bat d (3.11)]$ git checkout fce8054f4887fbe36e
...
[chris@bat d (fce8054f...)]$ make  # crée venv/cpython et extraie cpython ...
...

Je dois être dans la configuration avant cassage. Cassons !

[chris@bat d (fce8054f...)] python .scripts/merge.py afa24d52b821c2020
La position précédente de HEAD était sur 2e0ead5f22 ...
HEAD est maintenant sur afa24d52b8 ...
0 new files.
496 files to update.
merging pot files: 100%                   496/496
0 removed files.
Cleaning rst path in pot files: 100% 496/496
Fixing wrapping of po files: 100%    496/496
[HEAD détachée 79811b2d] Make merge
 319 files changed, 24916 insertions(+), 14531 deletions(-)

À ce moment library/shutil.po ne devrait différer avec upstream que pour la ligne "POT-Creation-Date:"et éventuellement du formatage powrap. Vérifions :

(79811b2d...)] git diff remote/origin -- library/shutil.po
diff --git a/library/shutil.po b/library/shutil.po
  diff --git a/library/shutil.po b/library/shutil.po
  index c50c465e..ae5d4e61 100644
  --- a/library/shutil.po
  +++ b/library/shutil.po
  @@ -5,7 +5,7 @@ msgid ""
   msgstr ""
   "Project-Id-Version: Python 3\n"
   "Report-Msgid-Bugs-To: \n"
  -"POT-Creation-Date: 2023-07-23 14:38+0200\n"
  +"POT-Creation-Date: 2023-09-23 20:09+0200\n"
   "PO-Revision-Date: 2018-11-29 18:26+0100\n"
   "Last-Translator: FULL NAME <EMAIL@ADDRESS>\n"
   "Language-Team: FRENCH <traductions@lists.afpy.org>\n"
[...] 
   #: library/shutil.rst:658
   msgid ""
  -"The keyword-only *filter* argument, which was added in Python 3.11.4, is "
  -"passed to the underlying unpacking function. For zip files, *filter* is not "
  -"accepted. For tar files, it is recommended to set it to ``'data'``, unless "
  -"using features specific to tar and UNIX-like filesystems. (See :ref:`tarfile-"
  -"extraction-filter` for details.) The ``'data'`` filter will become the "
  -"default for tar files in Python 3.14."
  -msgstr ""
  -
  -#: library/shutil.rst:652
  -msgid ""
   "Raises an :ref:`auditing event <auditing>` ``shutil.unpack_archive`` with "
   "arguments ``filename``, ``extract_dir``, ``format``."
   msgstr ""
[chris@bat d ((79811b2d...))]$

Ma conclusion : je n'arrive pas à reproduire le problème. Le msgid en dessous de #: library/shutil.rst:658 est un bug. Il doit être supprimé.

J'essaie de reproduire le « cassé » : ``` [chris@bat tmp]$ git clone https://git.afpy.org/AFPy/python-docs-fr.git d && cd d [chris@bat d]$ grep '^CPYTHON_CURRENT_COMMIT' Makefile # future destination CPYTHON_CURRENT_COMMIT := afa24d52b821c2020e0966f96a6205bca7db85e9 [chris@bat d (3.11)]$ git log | head -n 40 # trouve une rev avant cassage ... [chris@bat d (3.11)]$ git checkout fce8054f4887fbe36e ... [chris@bat d (fce8054f...)]$ make # crée venv/cpython et extraie cpython ... ... ``` Je dois être dans la configuration avant cassage. Cassons ! ``` [chris@bat d (fce8054f...)] python .scripts/merge.py afa24d52b821c2020 La position précédente de HEAD était sur 2e0ead5f22 ... HEAD est maintenant sur afa24d52b8 ... 0 new files. 496 files to update. merging pot files: 100% 496/496 0 removed files. Cleaning rst path in pot files: 100% 496/496 Fixing wrapping of po files: 100% 496/496 [HEAD détachée 79811b2d] Make merge 319 files changed, 24916 insertions(+), 14531 deletions(-) ``` À ce moment ``library/shutil.po`` ne devrait différer avec ``upstream`` que pour la ligne ``"POT-Creation-Date:"``et éventuellement du formatage powrap. Vérifions : ``` (79811b2d...)] git diff remote/origin -- library/shutil.po diff --git a/library/shutil.po b/library/shutil.po diff --git a/library/shutil.po b/library/shutil.po index c50c465e..ae5d4e61 100644 --- a/library/shutil.po +++ b/library/shutil.po @@ -5,7 +5,7 @@ msgid "" msgstr "" "Project-Id-Version: Python 3\n" "Report-Msgid-Bugs-To: \n" -"POT-Creation-Date: 2023-07-23 14:38+0200\n" +"POT-Creation-Date: 2023-09-23 20:09+0200\n" "PO-Revision-Date: 2018-11-29 18:26+0100\n" "Last-Translator: FULL NAME <EMAIL@ADDRESS>\n" "Language-Team: FRENCH <traductions@lists.afpy.org>\n" [...] #: library/shutil.rst:658 msgid "" -"The keyword-only *filter* argument, which was added in Python 3.11.4, is " -"passed to the underlying unpacking function. For zip files, *filter* is not " -"accepted. For tar files, it is recommended to set it to ``'data'``, unless " -"using features specific to tar and UNIX-like filesystems. (See :ref:`tarfile-" -"extraction-filter` for details.) The ``'data'`` filter will become the " -"default for tar files in Python 3.14." -msgstr "" - -#: library/shutil.rst:652 -msgid "" "Raises an :ref:`auditing event <auditing>` ``shutil.unpack_archive`` with " "arguments ``filename``, ``extract_dir``, ``format``." msgstr "" [chris@bat d ((79811b2d...))]$ ``` Ma conclusion : je n'arrive pas à reproduire le problème. Le msgid en dessous de ``#: library/shutil.rst:658`` est un bug. Il doit être supprimé.

ygingras commented

2023-11-23 16:05:55 +00:00

Après investigation, en effet, le PO ne correspond pas au .rst original. Le message du rst de la ligne 649 est dupliqué et est repris dans le .po avec l'id 658, ce qui est inexact. Le message suivant, qui est listé avec la ligne 652 est en fait à la ligne 658 dans le .rst.

La correction proposé par Christophe est bonne, mais il faut aussi corriger la ligne suivante.

Après investigation, en effet, le PO ne correspond pas au .rst original. Le message du rst de la ligne 649 est dupliqué et est repris dans le .po avec l'id 658, ce qui est inexact. Le message suivant, qui est listé avec la ligne 652 est en fait à la ligne 658 dans le .rst. La correction proposé par Christophe est bonne, mais il faut aussi corriger la ligne suivante.

ChristopheNan requested changes 2023-08-23 12:06:38 +00:00

ChristopheNan left a comment

Erreur de CI

deronnax force-pushed library_shutil from 9cbe90c337 to 89b9f17976

2023-08-27 09:49:01 +00:00

Compare

deronnax requested review from ChristopheNan 2023-08-27 10:10:41 +00:00

deronnax commented

2023-08-27 10:12:38 +00:00

Hop, erreur de CI corrigée. Il y a un bug Gitea qui marque le dernier commit en rouge mais le dernier commit (que j'ai force push et qui semble avoir rendu confus Gitea) est bien vert ✅.

ChristopheNan reviewed 2023-08-27 14:10:20 +00:00

library/shutil.po Outdated

 @ -125,6 +126,8 @@ msgid ""
 "Raises an :ref:`auditing event <auditing>` ``shutil.copyfile`` with "
 "arguments ``src``, ``dst``."
 msgstr ""
 "Lève un événement :ref:`événement d’audit <auditing>` ``shutil.copyfile`` "

ChristopheNan commented

2023-08-27 14:10:20 +00:00

événement est dupliqué.
s/Lève un événement/Lève un/

*événement* est dupliqué. s/Lève un événement/Lève un/

deronnax marked this conversation as resolved

ChristopheNan requested changes 2023-08-27 14:17:31 +00:00

library/shutil.po Outdated

 @ -183,3 +189,4 @@
 "Raises an :ref:`auditing event <auditing>` ``shutil.copymode`` with "
 "arguments ``src``, ``dst``."
 msgstr ""
 "Lève un événement :ref:`événement d’audit <auditing>` ``shutil.copymode`` "

ChristopheNan commented

2023-08-27 14:11:11 +00:00

Idem remarque ci-dessus.

deronnax marked this conversation as resolved

deronnax added 1 commit 2023-08-27 15:20:56 +00:00

ci/woodpecker/pr/woodpecker Pipeline was successful Details

939c8b182e

supprime doublon

deronnax commented

2023-08-27 15:21:13 +00:00

ah, bien vu. Pardon

jmdev reviewed 2023-11-19 16:13:24 +00:00

eviau reviewed 2023-11-19 16:13:53 +00:00

library/shutil.po

						
				@ -88,0 +89,4 @@

				"Copie le contenu (sans métadonnées) du fichier nommé *src* dans un fichier "

				"nommé *dst* et renvoie *dst* de la manière la plus efficace possible. *src* "

				"et *dst* sont des objets de type chemin ou des noms de chemin donnés sous "

				"forme de chaînes de caractères."

eviau commented

2023-11-19 16:13:52 +00:00

Afin de commencer une phrase par un mot en langue humaine... suggestion:

"Copie le contenu (sans métadonnées) du fichier nommé src dans un fichier "
"nommé dst et renvoie dst de la manière la plus efficace possible. Les objets src "
"et dst sont de type chemin ou des noms de chemin donnés sous "
"forme de chaînes de caractères."

Afin de commencer une phrase par un mot en langue humaine... suggestion: > "Copie le contenu (sans métadonnées) du fichier nommé *src* dans un fichier " "nommé *dst* et renvoie *dst* de la manière la plus efficace possible. Les objets *src* " "et *dst* sont de type chemin ou des noms de chemin donnés sous " "forme de chaînes de caractères."

eviau commented

2023-11-19 16:19:29 +00:00

C'est vraiment une suggestion perso, je voulais montrer comment faire des commentaires sur cette interface dans le cadre du sprint de traduction :)

jmdev commented

2023-11-19 17:34:11 +00:00

First-time contributor

Suggestion:

"Copie le contenu (sans métadonnées) du fichier nommé src dans un fichier "
"nommé dst et renvoie dst de la manière la plus efficace possible. Les paramètres"
src et dst sont des :term:objets simili-chemin <path-like object> ou des chemins
"donnés sous forme de chaînes."

Dans la doc on utilise généralement "chemin" pour traduire "pathname" et "chaîne" pour traduire "string", ça allège la lecture.
Selon le glossaire, on utilise "objet simili-chemin" pour traduire path-like object. J'ai essayé de mettre
le lien au glossaire, mais je ne suis pas 100% sûr d'avoir réussi.
J'ai mis "paramètres" plutôt que "objets" en adaptant la suggestion d'Édith.

Suggestion: > "Copie le contenu (sans métadonnées) du fichier nommé *src* dans un fichier " "nommé *dst* et renvoie *dst* de la manière la plus efficace possible. Les paramètres" *src* et *dst* sont des :term:`objets simili-chemin <path-like object>` ou des chemins "donnés sous forme de chaînes." 1. Dans la doc on utilise généralement "chemin" pour traduire "pathname" et "chaîne" pour traduire "string", ça allège la lecture. 2. Selon le glossaire, on utilise "objet simili-chemin" pour traduire path-like object. J'ai essayé de mettre le lien au glossaire, mais je ne suis pas 100% sûr d'avoir réussi. 3. J'ai mis "paramètres" plutôt que "objets" en adaptant la suggestion d'Édith.

👍 1