- Notifications
You must be signed in to change notification settings - Fork266
Makefile: don't rungit checkout - at the end of the build#1619
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 ourterms of service andprivacy statement. We’ll occasionally send you account related emails.
Already on GitHub?Sign in to your account
Uh oh!
There was an error while loading.Please reload this page.
Conversation
PyDocTeur commentedMay 4, 2021
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 del'AFPy et de Traduction I'm a bot made by theTranslation 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. |
PyDocTeur commentedMay 4, 2021
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 del'AFPy et de Traduction I'm a bot made by theTranslation and AFPy teams on their free (state: incorrect_title) |
1 similar comment
PyDocTeur commentedMay 4, 2021
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 del'AFPy et de Traduction I'm a bot made by theTranslation 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 |
PyDocTeur commentedMay 4, 2021
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 del'AFPy et de Traduction I'm a bot made by theTranslation 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.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others.Learn more.
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.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others.Learn more.
Juste pour être sûr. Tu parles de garder la lignegit checout $(CPYTHON_CURRENT_COMMIT) mais d'enlever legit checkout - ?
C'est une solution possible. C'est juste que par rapport à d'autres projets similaires, ça me surprend moins d'avoir unmake qui me dit que mon code est pas à jour qu'unmake qui lancegit à ma place, mais ça reste un argument subjectif.
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.
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 dansensure_prerequisites que son repo est clean.
PyDocTeur commentedMay 5, 2021
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 del'AFPy et de Traduction I'm a bot made by theTranslation 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 slowerbecause many sources may seem out of date.
git checkout - at the end of the buildPyDocTeur commentedMay 12, 2021
Rajoutez-moi un DisclaimerJe suis un robot fait par l'équipe del'AFPy et de Traduction I'm a bot made by theTranslation 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 runany git operations if there's nothing to be done