Skip to content

Howto ipaddress.po#1870

Closed
ghost wants to merge 22 commits into
3.10from
unknown repository
Closed

Howto ipaddress.po#1870
ghost wants to merge 22 commits into
3.10from
unknown repository

Conversation

@ghost

@ghost ghost commented May 12, 2022

Copy link
Copy Markdown

Quelques fuzzies.

Comment thread howto/ipaddress.po Outdated
Comment on lines +267 to +269
"Vous vous êtes donné la peine de créer un objet adresse (réseau ou "
"interface) IPv4 ou IPv6 ; le module :mod:`ipaddress` permet d'obtenir des "
"informations sur celui-ci de façon facile et intuitive."

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Avec cette traduction, on ne comprend plus ce que l'auteur veut dire par « Vous vous êtes donné la peine… ».

Suggested change
"Vous vous êtes donné la peine de créer un objet adresse (réseau ou "
"interface) IPv4 ou IPv6 ; le module :mod:`ipaddress` permet d'obtenir des "
"informations sur celui-ci de façon facile et intuitive."
"Si vous vous êtes donné la peine de créer un objet IPv4 ou Ipv6, que ce soit une adresse, "
"un réseau, ou une interface, c'est sûrement pour en tirer quelque chose. Le module :mod:`ipaddress` permet d'obtenir des "
"informations sur cet objet de façon facile et intuitive."

Après, il y a aussi la possibilité de ne pas conserver cette tournure très informelle.

Note aussi qu'il ne me semble pas que l'original aie le sens de « adresse (réseau ou interface) » mais de « adresse, réseau, ou interface ».

@jeanas jeanas mentioned this pull request May 12, 2022
@jeanas

jeanas commented May 16, 2022

Copy link
Copy Markdown
Collaborator

@th71 La PR #1869 a fait des changements dans le même fichier, donc il va falloir résoudre les conflits de fusion. Pour cela :

git checkout 3.10
git pull
git checkout howto-ipaddress.po
git merge 3.10

À ce stade, Git devrait t'avertir sur un conflit dans howto/ipaddress.po. Ouvre le fichier dans un éditeur de texte pur (pas dans Poedit, il ne comprendra pas ce qui se passe), et examine les parties marquées

<<<<<<<<<<<<<<<<<<<
...
====================
...
>>>>>>>>>>>>>>>>>>>>>

(les chevrons sont peut-être dans l'autre sens, je ne sais plus). À chaque fois, il faut remplacer cela par une version qui intègre les changements des deux côtés ou qui choisit juste l'un des deux. N'hésite pas à demander de l'aide.

@ghost

ghost commented May 16, 2022

Copy link
Copy Markdown
Author

Fait.

@ghost

ghost commented May 16, 2022

Copy link
Copy Markdown
Author

Je pense que l'on peut clore cette PR pour l'instant; je ferai un relecture à un autre moment, inutile de bloquer le fichier pour rien.

@jeanas

jeanas commented May 16, 2022

Copy link
Copy Markdown
Collaborator

Comme tu veux, moi je ne trouve pas ça dérangeant que les PR restent un peu de temps ouvertes (celle-ci ne l'est que depuis 4 jours).

@jeanas

jeanas commented May 16, 2022

Copy link
Copy Markdown
Collaborator

Par contre, tu as encore ressuscité logging.po.orig 😄

@ghost

ghost commented May 16, 2022

Copy link
Copy Markdown
Author

Alors là je ne comprend pas... Je suis en train de me demander si les fichiers .orig ne sont pas générés par l'outil de fusion que j'utilise. (meld)

@jeanas

jeanas commented May 16, 2022

Copy link
Copy Markdown
Collaborator

Ah, une recherche suggère

git config --global mergetool.keepBackup false

@ghost

ghost commented May 16, 2022

Copy link
Copy Markdown
Author

Fait, j'espère que ça règlera le problème définitivement.

@jeanas

jeanas commented May 16, 2022

Copy link
Copy Markdown
Collaborator

Tu as bien enlevé le fichier, mais tu peux voir dans la partie « diff » de la PR qu'il reste des changements non-voulus à library/logging.po et whatsnew/3.10.po. Sur ta branche, la commande

git log --graph -- library/logging.po

me donne

*   commit bd3b555ad810088370a5d054fc8ef0fd9785b638
|\  Merge: 32ae6bb9 b8eb0ae2
| | Author: = <abonnements@thierry-pelle.eu>
| | Date:   Tue Apr 5 18:02:07 2022 +0200
| | 
| |     Merge branch '3.10' of https://github.com/python/python-docs-fr into 3.10
| | 
| * commit b8eb0ae2077a029a4d7e86f9d88b508216aaef4c
| | Author: Julien Palard <julien@palard.fr>
| | Date:   Tue Apr 5 12:13:01 2022 +0200
| | 
| |     Make merge 3.10 (#1837)
| |     
| |     * Make merge
| |     
| |     * Whine about lines too long only if line contains a space.
| | 
* | commit 32ae6bb9a82d67b3e980da92856969c4883677cd
|\| Merge: 307d919f 518292ad
| | Author: = <abonnements@thierry-pelle.eu>
| | Date:   Wed Mar 30 12:03:31 2022 +0200
| | 
| |     Fusion depuis upstream
| | 
| * commit 518292adbb4b26915a5d54a96271bf4c00ee975f
| | Author: Julien Palard <julien@palard.fr>
| | Date:   Fri Mar 18 17:07:39 2022 +0100
| | 
| |     Make merge
| | 
* | commit 307d919f1f2177358806fc92e44d4b0896d44e47
|/  Author: = <abonnements@thierry-pelle.eu>
|   Date:   Wed Mar 23 17:16:19 2022 +0100
|   
|       Modif loggin
| 
* commit 2479b6ba064e09634be6d643edaebe2823596dfc
| Author: Julien Palard <julien@palard.fr>
| Date:   Fri Dec 31 11:41:52 2021 +0100
| 
|     Make merge (#1799)

À en juger par le titre "Merge branch '3.10' of https://github.com/python/python-docs-fr into 3.10", tu dois avoir ces changements dans ta branche 3.10 locale. Tu n'avais pas fait #1864 (comment) ? Ou bien tu as commencé cette PR avant de le faire ?

@ghost

ghost commented May 16, 2022

Copy link
Copy Markdown
Author

Il me semble que j'avais essayé de partir d'une base propre (puisque je pensais avoir compris le problème).

@jeanas

jeanas commented May 16, 2022

Copy link
Copy Markdown
Collaborator

Manifestement la base n'était pas propre.

Tu veux que je refasse la manip pour nettoyer ?

@ghost

ghost commented May 16, 2022

Copy link
Copy Markdown
Author

Ben si j'ai bien fait #1864 ce qui avait résolu le problème à l'époque.

@ghost

ghost commented May 16, 2022

Copy link
Copy Markdown
Author

Je vais essayer de nouveau.

@ghost

ghost commented May 20, 2022

Copy link
Copy Markdown
Author

Manifestement la base n'était pas propre.

Tu veux que je refasse la manip pour nettoyer ?

Bon je viens de reprendre la manip que tu me suggérait mais je ne vois pas la suppression des fichiers qui ne devraient pas être là (bon tu va dire je n'ai fait de recopier sans forcément comprendre j'ai sûrement dû oublier qqchose).

Si tu as le temps pour faire la manip de nettoyage merci (ensuite je repartirai d'un dépôt local neuf, ça devrait éviter ce problème récurrent).

@jeanas

jeanas commented May 20, 2022

Copy link
Copy Markdown
Collaborator

Pas de problème, par contre, ce qui est plus troublant, c'est que je ne vois plus aucun changement sur howto/ipaddress.po dans cette PR… Les changements de #1869 ont complètement recouvert les tiens ?

@ghost

ghost commented May 20, 2022

Copy link
Copy Markdown
Author

Normalement aucune différence avec la version précédente de howto/ipaddress.po, j'ai seulement vainement essayé de supprimer les traces indésirables. Ou alors c'est un effet de bord...

@jeanas

jeanas commented May 21, 2022

Copy link
Copy Markdown
Collaborator

@th71 L'historique Git de cette PR est tellement compliqué que je suis complètement perdu…

Honnêtement, à mon avis, le plus simple est de la fermer et de refaire les changements, puis de les soumettre dans une nouvelle PR.

@ghost

ghost commented May 21, 2022

Copy link
Copy Markdown
Author

Tout à fait d'accord avec toi.
C'est parti!
Espérons que la suite sera plus propre :-)

@ghost ghost closed this May 21, 2022
@jeanas

jeanas commented May 21, 2022

Copy link
Copy Markdown
Collaborator

Très bien. Pense bien à partir d'une branche 3.10 propre : git switch 3.10; git pull (et normalement cela ne devrait pas créer un commit « merge branch 3.10… » ; si c'est le cas dis-le).

This pull request was closed.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants