Conversation
entwanne
left a comment
There was a problem hiding this comment.
It's good for a first review, thank you.
I have post some comments (in french) and noticed a few errors.
extending/windows.po
Outdated
| "programmer interested in producing software which can be successfully built " | ||
| "on both Unix and Windows." | ||
| msgstr "" | ||
| "Ce paragraphe explique rapidement comment créer un module d'extension Windows " |
There was a problem hiding this comment.
Je n'aurais pas utilisé le terme « paragraphe », il me semble que c'est ici de toute la page qu'il est question.
extending/windows.po
Outdated
| "still need the C compiler that was used to build Python; typically Microsoft " | ||
| "Visual C++." | ||
| msgstr "" | ||
| "Les auteurs de modules sont invités à utiliser l'approche distutils pour " |
There was a problem hiding this comment.
Peut-être passer « distutils » en italique ?
extending/windows.po
Outdated
| "working with. For example, if you are using Python 2.2.1, ``XY`` will " | ||
| "actually be ``22``." | ||
| msgstr "" | ||
| "Ce paragraphe mentionne plusieurs noms de fichiers comprenant un numéro de " |
There was a problem hiding this comment.
Idem pour « paragraphe ».
extending/windows.po
Outdated
| msgstr "" | ||
| "Ce paragraphe mentionne plusieurs noms de fichiers comprenant un numéro de " | ||
| "version Python encodé. Ces noms de fichiers sont construits sous le format " | ||
| "de version "XY" ; en pratique, ``'X'`` représente le numéro de version majeure" |
There was a problem hiding this comment.
Pourquoi avoir remplacé «``XY``» par « "XY" » ? Je ne suis pas sûr que ça passe bien la compilation.
extending/windows.po
Outdated
| "Il y a deux approches lorsque l'on construit des modules d'extension sur " | ||
| "Windows, tout comme sur Unix : utiliser le paquet :mod:`distutils` pour " | ||
| "contrôler le processus de construction, ou faire les choses manuellement." | ||
| "L'approche distutils fonctionne bien pour la plupart des extensions; la " |
There was a problem hiding this comment.
Italique ?
+ espace (insécable) avant le point virgule.
extending/windows.po
Outdated
| "Supposons que vous construisez deux modules de chargement dynamiques, B et C, " | ||
| "qui ne devraient pas partager un autre bloc de code avec A. Sur Unix, vous ne " | ||
| "transmettrez pas :file:`A.a` au lieur pour :file:`B.so` et :file:`C.so` ; cela" | ||
| "le ferrait être inclus deux fois, pour que B et C aient chacuns leur propre " |
There was a problem hiding this comment.
« ferrait » → « ferait »
extending/windows.po
Outdated
| "est comme un fichier Unix :file:`.a` ; elle contient du code pouvant être " | ||
| "inclus si nécessaire. Une librairie d'import est uniquement utilisée pour " | ||
| "rassurer le lieur qu'un certain identificateur est légal, et sera présent dans" | ||
| "le programme quand le DLL est chargé. Comme ça le lieur utilise les " |
There was a problem hiding this comment.
Le masculin sur « DLL » c'est pour « fichier DLL » ? Je trouve ça un peu déroutant.
extending/windows.po
Outdated
| "or may not work (though Borland seems to). The rest of this section is MSVC+" | ||
| "+ specific." | ||
| msgstr "" | ||
| "Windows Python est construit en Microsoft Visual C++ ; utiliser d'autres " |
There was a problem hiding this comment.
« Windows Python » → « Le Python de Windows » ?
extending/windows.po
Outdated
| msgstr "" | ||
| "Windows Python est construit en Microsoft Visual C++ ; utiliser d'autres " | ||
| "compilateurs pourrait fonctionner, ou pas (cependant Borland a l'air de " | ||
| "fonctionner). The reste de cette section est spécifique à MSVC++." |
extending/windows.po
Outdated
| "the Project Settings dialog, Link tab, to specify *ignore default " | ||
| "libraries*. Add the correct :file:`msvcrtxx.lib` to the list of libraries." | ||
| msgstr "" | ||
| "Le studio de développeur apportera beaucoup de librairies d'import dont vous" |
There was a problem hiding this comment.
C'est bizarre ce « Developer Studio » avec les majuscules, ça fait plutôt nom de programme, je ne suis pas sûr que ce soit à traduire.
|
Bonsoir, merci pour la mise en évidence des coquilles et traductions maladroites! |
extending/windows.po
Outdated
| "Cette page explique rapidement comment créer un module d'extension Windows " | ||
| "pour Python en utilisant Microsoft Visual C++, et donne plus d'informations " | ||
| "contextuelles sur son fonctionnement. Le texte explicatif est utile tant pour " | ||
| "le programmateur Windows qui apprend à construire des extensions Python que " |
There was a problem hiding this comment.
« programmateur » → « développeur » (C'est pas écrit Gardena sur mon front).
extending/windows.po
Outdated
| "pour Python en utilisant Microsoft Visual C++, et donne plus d'informations " | ||
| "contextuelles sur son fonctionnement. Le texte explicatif est utile tant pour " | ||
| "le programmateur Windows qui apprend à construire des extensions Python que " | ||
| "pour le programmateur Unix souhaitant produire des logiciels pouvant être " |
There was a problem hiding this comment.
Pareil, je préfère développeur
extending/windows.po
Outdated
| "library module." | ||
| msgstr "" | ||
| "Il y a deux approches lorsque l'on construit des modules d'extension sur " | ||
| "Windows, tout comme sur Unix : utiliser le paquet *:mod:`distutils`* pour " |
There was a problem hiding this comment.
Comme le disait Antoine, il faut mettre les noms en italique, mais là avec le tag :mod: il est déjà visuellement bien différent du reste, donc là, pas besoin d'italique, désolé pour le ping-pong :)
extending/windows.po
Outdated
| "Il y a deux approches lorsque l'on construit des modules d'extension sur " | ||
| "Windows, tout comme sur Unix : utiliser le paquet *:mod:`distutils`* pour " | ||
| "contrôler le processus de construction, ou faire les choses manuellement." | ||
| "L'approche distutils fonctionne bien pour la plupart des extensions ; la " |
There was a problem hiding this comment.
Il semble manquer un espace après le point, avant la majuscule.
| "Sur Unix, un fichier objet partagé (:file:`.so`) contient du code servant au " | ||
| "programme, ainsi que les noms des fonctions et les données que l'on s'attend " | ||
| " à trouver dans le programme. Quand le fichier est attaché au programme, " | ||
| "toutes les réferences à ces fonctions et données dans le code du fichier sont" |
There was a problem hiding this comment.
"sontmodifiés" il manque un espace, quel éditeur utilise-tu ? Vé ? Si tu utilise Vé il y a un bug sur les espaces entre chaînes, je te conseille soit d'aller réparer Vé, soit de l'abandonner.
There was a problem hiding this comment.
Notepad++ haha ¯_(ツ)_/¯
extending/windows.po
Outdated
| msgstr "" | ||
| "Sur Windows, il y a deux types de bibliothèques, une bibliothèque statique et une " | ||
| "bibliothèque d'import (toutes deux appelées :file:`.lib`). Une bibliothèque statique" | ||
| "est comme un fichier Unix :file:`.a` ; elle contient du code pouvant être " |
There was a problem hiding this comment.
« statiqueest » il manque un espace.
extending/windows.po
Outdated
| "est comme un fichier Unix :file:`.a` ; elle contient du code pouvant être " | ||
| "inclus si nécessaire. Une bibliothèque d'import est uniquement utilisée pour " | ||
| "rassurer le lieur qu'un certain identificateur est légal, et sera présent dans" | ||
| "le programme quand la DLL est chargé. Comme ça le lieur utilise les " |
extending/windows.po
Outdated
| "Supposons que vous construisez deux modules de chargement dynamiques, B et C, " | ||
| "qui ne devraient pas partager un autre bloc de code avec A. Sur Unix, vous ne " | ||
| "transmettrez pas :file:`A.a` au lieur pour :file:`B.so` et :file:`C.so` ; cela" | ||
| "le ferait être inclus deux fois, pour que B et C aient chacuns leur propre " |
extending/windows.po
Outdated
| "La première commande a créé trois fichiers : :file:`spam.obj`, :file:`spam." | ||
| "dll` et :file:`spam.lib`. :file:`Spam.dll` ne contient pas de fonctions Python" | ||
| " (telles que :c:func:`PyArg_ParseTuple`), mais il sait comment trouver le code" | ||
| "Python grâce à :file:`pythonXY.lib`." |
| "the Project Settings dialog, Link tab, to specify *ignore default " | ||
| "libraries*. Add the correct :file:`msvcrtxx.lib` to the list of libraries." | ||
| msgstr "" | ||
| "Developer Studio apportera beaucoup de bibliothèques d'import dont vous " |
There was a problem hiding this comment.
« Developer Studio » en italique ?
|
Quel éditeur utilise-tu ? Je suis surpris des problèmes de concaténations entre lignes, ça me rappelle Vé, ou alors tu édite ça à la main ? Si tu les édite à la main, je te conseille de plutôt utiliser poedit :) |
|
Merci pour les retour et conseils vis à vis de l'éditeur, je vais jeter un coup d’œil à poedit! :) |
extending/windows.po
Outdated
| "contextuelles sur son fonctionnement. Le texte explicatif est utile tant pour " | ||
| "le développeur Windows qui apprend à construire des extensions Python que " | ||
| "pour le développeur Unix souhaitant produire des logiciels pouvant être " | ||
| "construits sur Unix et Windows |
There was a problem hiding this comment.
Il manque un point final et le guillemet fermant.
|
@JulienPalard , petite question : Je vois que des changements sont requis de ta part, mais je n'arrive pas à voir quoi (cf image)... Est-ce toujours d'actualité ou est-ce un commentaire qui a déjà été pris en compte? Merci, Valérian |
|
C'est bon pour moi, et c'est mergé. Attention à ne pas réutiliser ta branche, j'ai squash, 3.6 à donc divergé en intégrant tes modifications. Tu devrais (comme toujours en fait) checkout la 3.6, pull pour te mettre à jour, et créer une nouvelle branche à partir de là. Tu peux supprimer ta branche, elle ne sera plus utile. |
|
Merci! Par contre, je viens de voir ton mail, et entre temps j'ai fait une nouvelle PR pour index.po , qui je présume doit présenter le même problème du coup...:/ |
Closes #66 Co-authored-by: Mathias Laurin <Mathias.Laurin+github.com@gmail.com> Reviewed-on: https://git.afpy.org/AFPy/python-docs-fr/pulls/88 Co-authored-by: Synss <mathias.laurin+afpy.org@gmail.com> Co-committed-by: Synss <mathias.laurin+afpy.org@gmail.com>

First PR to check if I'm doing things right.
Just modified one document so far, extending>windows.po