Skip to content

Pas de -W explicite pour Sphinx#1680

Merged
JulienPalard merged 1 commit intopython:3.10from
jeanas:makefile-error-handling
Sep 6, 2021
Merged

Pas de -W explicite pour Sphinx#1680
JulienPalard merged 1 commit intopython:3.10from
jeanas:makefile-error-handling

Conversation

@jeanas
Copy link
Copy Markdown
Collaborator

@jeanas jeanas commented Sep 4, 2021

CPython ajoute déjà -W dans son Makefile. Depuis le commit
f98c1623ec90508937afc1b58556e38214d70892 ("Doc: -W flag for sphinx-build
can be disabled" par Julien Palard), c'est même configurable à l'aide de
la variable SPHINXERRORHANDLING. Avec le présent commit, cette variable
fonctionne aussi dans notre traduction, ce qui permet des recompilations
plus rapides pendant la traduction avec « make SPHINXERRORHANDLING= » ou
bien « make SPHINXERRORHANDLING="-W --keep-going" ». Ce serait bien
d'avoir --keep-going par défaut, mais je ne l'ajoute pas ici car il
serait sûrement encore mieux de le mettre dans CPython directement.

@JulienPalard
Copy link
Copy Markdown
Member

Retirer le -W accélère la compilation ?

@jeanas
Copy link
Copy Markdown
Collaborator Author

jeanas commented Sep 6, 2021

Non (enfin pas que je sache), mais lorsque l'on a traduit une vingtaine de chaînes et que l'on veut vérifier, on obtient souvent plusieurs avertissements -- c'est du moins mon cas ! -- , notamment celui sur les références incohérentes. Je trouve plus rapide de les voir tous et de corriger plutôt que d'en corriger un à chaque fois et d'attendre le suivant dans un nouveau tour de compilation. (Certes, les recompilations seront déjà plus rapides avec #1630 ou #1675, mais il n'empêche, et puis cette PR ne change pas la valeur par défaut, elle donne juste une possibilité supplémentaire.)

@jeanas
Copy link
Copy Markdown
Collaborator Author

jeanas commented Sep 6, 2021

Le message du commit n'était pas franchement clair, je l'ai édité un peu.

CPython ajoute déjà -W dans son Makefile. Depuis le commit
f98c1623ec90508937afc1b58556e38214d70892 ("Doc: -W flag for sphinx-build
can be disabled" par Julien Palard), c'est même configurable à l'aide de
la variable SPHINXERRORHANDLING. Avec le présent commit, cette variable
fonctionne aussi dans notre traduction, ce qui facilite la traduction
car cela permet d'obtenir tous les avertissements en une fois avec
« make SPHINXERRORHANDLING= » ou bien « make SPHINXERRORHANDLING="-W
--keep-going" », au lieu de voir un nouvel avertissement arrêter la
compilation à chaque fois. Ce serait bien d'avoir --keep-going par
défaut, mais je ne l'ajoute pas ici car il serait sûrement encore mieux
de le mettre dans CPython directement.
@PyDocTeur
Copy link
Copy Markdown

Hello @jean-abou-samra ! Désolé, mais ton titre de pull request me semble invalide par rapport à ce que je suis programmé d'accepter.
Merci de le corriger ou d'ajouter le label meta si c'est une PR spéciale. Un exemple de titre valide serait : « Traduction de dossier/fichier.po ».


Disclaimer

Je suis un robot fait par l'équipe de l'AFPy et de Traduction
sur leur temps libre. Je risque de dire des bétises. Ne me blâmez pas, blamez les développeurs.

Code source

I'm a bot made by the Translation and AFPy teams on their free
time. I might say or do dumb things sometimes. Don't blame me, blame the developer !

Source code

(state: incorrect_title)
PyDocTeur v1.12.0

@JulienPalard JulienPalard merged commit 28f90d0 into python:3.10 Sep 6, 2021
@JulienPalard
Copy link
Copy Markdown
Member

OK LGTM :)

@jeanas jeanas deleted the makefile-error-handling branch September 6, 2021 09:12
@jeanas
Copy link
Copy Markdown
Collaborator Author

jeanas commented Sep 6, 2021

Merci !

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