-
Notifications
You must be signed in to change notification settings - Fork 266
Trying to simplify the Makefile #1499
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
Trying to simplify the Makefile #1499
Conversation
@@ -688,7 +688,7 @@ msgstr "``n`` (:class:`int`) [``Py_ssize_t``]" | |||
|
|||
#: c-api/arg.rst:292 | |||
msgid "Convert a Python integer to a C :c:type:`Py_ssize_t`." | |||
msgstr "Convertit un entier Python en un :c:type:`short int` C." | |||
msgstr "Convertit un entier Python en un :c:type:`Py_ssize_t`." |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Spotted this one while playing with the Makefile.
Plus que la relecture, et si c'est bon, je merge ! DisclaimerJe suis un robot fait par l'équipe de l'AFPy et de Traduction I'm a bot made by the Translation and AFPy teams on their free (state: automerge) |
Oui, là c'est ballot 😢 ! Il devrait mettre un |
Makefile
Outdated
$(MODE) | ||
# Cleanup | ||
git -C $(CPYTHON_PATH) checkout - | ||
rm -fr $(CPYTHON_PATH)/locales/ |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Sûr ? On perd l'intérêt du cp -u
au-dessus si on recrée le dossier à chaque fois.
Ou bien le make doc se base sur la date de la dernière modification du fichier (nouvellement copié ou pas) et dans ce cas c'est le -u
qui devient superflu ?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Je rejoins @awecx : je souhaite garder l'arborescence.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
J'entend.
J'ai poussé une nouvelle proposition qui évite d'avoir a supprimer locales/
tout en évitant de 'poluer" l'arborescence du checkout de cpython, ça simplifie le git status
! cool.
C'est une tentative de simplifier le Makefile pour le rendre plus maintenable.
Alors forcément il fait un peu moins de choses, il n'essaye pas de tout automatiser et laisse par exemple au soin de l'utilisateur de créer son venv (comme il le souhaite, c'est pas plus mal) et d'y installer les dépendances.
Pour ce faire, le Makefile l'aide quand même :
La grosse partie supprimée est surtout la gestion des worktree, ça m'a probablement semblé une bonne idée le jour où j'ai fait ça, mais c'était probablement trop complexe pour être maintenu surtout dans un repo de trad'.
Mon envie de simplifier ce Makefile vient d'une erreur d'un membre de l'équipe qui, face a un bug dans le Makefile, à voulu exécuter les commandes à la main, et a copié collé :
Sauf qu'il n'avait pas déclaré la variable WORKTREES ni la variable BRANCH (et puis c'est de la syntaxe Makefile, pas bash), il a donc eu :
(qui exécute quand même
rm -fr /
derrière, et sur sa distrib il n'y a pas de--preserve-root
par défaut)RIP son home.