Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Howto ipaddress.po #1870

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Closed
wants to merge 22 commits into from
Closed

Howto ipaddress.po #1870

wants to merge 22 commits into from

Conversation

ghost
Copy link

@ghost ghost commented May 12, 2022

Quelques fuzzies.

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
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
Copy link
Collaborator

jeanas commented May 16, 2022

@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
Copy link
Author

ghost commented May 16, 2022

Fait.

@ghost
Copy link
Author

ghost commented May 16, 2022

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
Copy link
Collaborator

jeanas commented May 16, 2022

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
Copy link
Collaborator

jeanas commented May 16, 2022

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

@ghost
Copy link
Author

ghost commented May 16, 2022

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
Copy link
Collaborator

jeanas commented May 16, 2022

Ah, une recherche suggère

git config --global mergetool.keepBackup false

@ghost
Copy link
Author

ghost commented May 16, 2022

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

@jeanas
Copy link
Collaborator

jeanas commented May 16, 2022

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: = <[email protected]>
| | 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 <[email protected]>
| | 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: = <[email protected]>
| | Date:   Wed Mar 30 12:03:31 2022 +0200
| | 
| |     Fusion depuis upstream
| | 
| * commit 518292adbb4b26915a5d54a96271bf4c00ee975f
| | Author: Julien Palard <[email protected]>
| | Date:   Fri Mar 18 17:07:39 2022 +0100
| | 
| |     Make merge
| | 
* | commit 307d919f1f2177358806fc92e44d4b0896d44e47
|/  Author: = <[email protected]>
|   Date:   Wed Mar 23 17:16:19 2022 +0100
|   
|       Modif loggin
| 
* commit 2479b6ba064e09634be6d643edaebe2823596dfc
| Author: Julien Palard <[email protected]>
| 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
Copy link
Author

ghost commented May 16, 2022

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

@jeanas
Copy link
Collaborator

jeanas commented May 16, 2022

Manifestement la base n'était pas propre.

Tu veux que je refasse la manip pour nettoyer ?

@ghost
Copy link
Author

ghost commented May 16, 2022

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

@ghost
Copy link
Author

ghost commented May 16, 2022

Je vais essayer de nouveau.

@ghost
Copy link
Author

ghost commented May 20, 2022

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
Copy link
Collaborator

jeanas commented May 20, 2022

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
Copy link
Author

ghost commented May 20, 2022

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
Copy link
Collaborator

jeanas commented May 21, 2022

@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
Copy link
Author

ghost commented May 21, 2022

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
Copy link
Collaborator

jeanas commented May 21, 2022

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