-
-
Notifications
You must be signed in to change notification settings - Fork 30.9k
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
gh-97731: fix distclean target to clean docs #97732
Conversation
I don't think a NEWS entry is required for this, but I don't see how to add a "skip news" label... |
Thanks @hugovk. "Labels" is inactive for me. I don't know where type-bug came from either. I guess that was the default for a bug report? |
Yes, type-bug is automatically added when using the bug template, defined here: https://github.com/python/cpython/blob/main/.github/ISSUE_TEMPLATE/bug.md |
This change looks good to me. I will let the build, release managers comment as it can serve or affect their work. |
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.
LGTM.
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.
Looks good, reduces duplication.
From the issue:
Doc/tools/{sphinx,pygments,docutils}
doesn' exist after executingmake -C Doc html
. I suspect thedocclean
target could simply be replaced bymake -C Doc clean
, and made a dependency of thedistclean
target.
Those dirs look redundant now.
Deletion of those dirs from the top-level Makefile.pre.in
added 15 years ago (8188e63).
Also 15 years ago, Doc/Makefile
used to only delete Docs/tools/sphinx
(116aa62).
But use of a venv was added 8 years ago (d5ea39d), so dependencies get wiped from the venv dir.
Thanks @smontanaro for the PR, and @orsenthil for merging it 🌮🎉.. I'm working now to backport this PR to: 3.10, 3.11. |
Sorry, @smontanaro and @orsenthil, I could not cleanly backport this to |
Sorry @smontanaro and @orsenthil, I had trouble checking out the |
The cherry_picker failed. Please approve and merge these backports |
This change makes the
distclean
target inMakefile.pre.in
rely on theclean
target inDoc/Makefile
to do that part of its job.