Makefile: don't run git checkout - at the end of the build
#1619
Conversation
|
Hello @dmerejkowsky ! Désolé, mais ton titre de pull request me semble invalide par rapport à ce que je suis programmé d'accepter. 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: incorrect_title) |
|
Note: j'ai d'autres idée pour rendre le build plus rapide - en tout cas pendant le développement, notamment:
mais ça me paraît un bon premier pas. |
|
Hello @dmerejkowsky ! Désolé, mais ton titre de pull request me semble invalide par rapport à ce que je suis programmé d'accepter. 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: incorrect_title) |
1 similar comment
|
Hello @dmerejkowsky ! Désolé, mais ton titre de pull request me semble invalide par rapport à ce que je suis programmé d'accepter. 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: incorrect_title) |
|
Quelques mesures. À chaque fois, on mesure le temps pris par Sans ce patch, si cpython est sur une autre branche (par exemple, le tag 3.9): 5 minutes Avec ce patch, en générant le fichier NEWS: 30s Avec ce patch, et sans générer le fichiers NEWS: 12s À noter pour le dernier cas : si on ne change aucun fichier |
|
Hello @dmerejkowsky ! Désolé, mais ton titre de pull request me semble invalide par rapport à ce que je suis programmé d'accepter. 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: incorrect_title) |
Makefile
Outdated
| git -C $(CPYTHON_PATH) checkout $(CPYTHON_CURRENT_COMMIT) | ||
| @if [ $(shell git -C $(CPYTHON_PATH) rev-parse HEAD) != $(CPYTHON_CURRENT_COMMIT) ]; then \ | ||
| echo "Your $(CPYTHON_PATH) repositoy is not at the currect revision"; \ | ||
| echo "You can run try and run "; \ | ||
| echo "git -C $(CPYTHON_PATH) checkout $(CPYTHON_CURRENT_COMMIT)"; \ | ||
| echo "to fix the problem"; \ | ||
| exit 1; \ | ||
| fi |
There was a problem hiding this comment.
Pourquoi pas laisser cette ligne comme elle était ? Elle me paraît bonne pour les raisons suivantes :
gitne touchera pas aux fichiers si on est déjà sur la bonne branche, donc il n'invalidera pas le cache de Sphinx- C'est plus cool pour l'utilisateur de ne pas avoir à le faire lui même.
- On ne peut pas "casser" le repo d'un dev cpython de toute façons car avant ça on vérifie que son repo est clean
Et plus moins il y a de lignes à maintenir mieux je me porte :D
There was a problem hiding this comment.
Juste pour être sûr. Tu parles de garder la ligne git checout $(CPYTHON_CURRENT_COMMIT) mais d'enlever le git checkout - ?
C'est une solution possible. C'est juste que par rapport à d'autres projets similaires, ça me surprend moins d'avoir un make qui me dit que mon code est pas à jour qu'un make qui lance git à ma place, mais ça reste un argument subjectif.
There was a problem hiding this comment.
Juste pour être sûr. Tu parles de garder la ligne
git checout $(CPYTHON_CURRENT_COMMIT)mais d'enlever legit checkout -?
Oui, car c'est sans risque, puisqu'on vérifie dans ensure_prerequisites que son repo est clean.
|
Hello @dmerejkowsky ! Désolé, mais ton titre de pull request me semble invalide par rapport à ce que je suis programmé d'accepter. 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: incorrect_title) |
Running `git checkout CPYTHON_CURRENT_COMMIT` and then `git checkout -c` can render the sphinx build slower because many sources may seem out of date.
b595bee to
778967a
Compare
git checkout - at the end of the build
|
Rajoutez-moi un 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: approved) |
Running
git checkout CPYTHON_CURRENT_COMMITand thengit checkout -ccan render the spinx build slowerbecause many sources may seem out of date.
It's better to check if we are at the correct revision and
not run any git operations if there's nothing to be done