Skip to content

[3.9] bpo-45866: pegen strips directory of "generated from" header (GH-29777) (GH-29792) #29797

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

Merged
merged 1 commit into from
Nov 26, 2021
Merged

[3.9] bpo-45866: pegen strips directory of "generated from" header (GH-29777) (GH-29792) #29797

merged 1 commit into from
Nov 26, 2021

Conversation

vstinner
Copy link
Member

@vstinner vstinner commented Nov 26, 2021

"make regen-all" now produces the same output when run from a
directory other than the source tree: when building Python out of the
source tree.

(cherry picked from commit 253b7a0)
(cherry picked from commit b6defde)

https://bugs.python.org/issue45866

… (GH-29792)

"make regen-all" now produces the same output when run from a
directory other than the source tree: when building Python out of the
source tree.

(cherry picked from commit 253b7a0)
(cherry picked from commit b6defde)
@@ -0,0 +1,4 @@
``make regen-all`` now produces the same output when run from a directory
other than the source tree: when building Python out of the source tree.
pegen now strips directory of the "generated by pygen from <FILENAME" header
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
pegen now strips directory of the "generated by pygen from <FILENAME" header
pegen now strips directory of the "generated by pygen from <FILENAME>" header

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Oh, that's a backport of a change which already landed in main and 3.10 branches :-( I prefer to keep the typo. The typo can be fixed in a separated PR, but I'm not really motivated to write a PR just for that :-p

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Well, I created PR to fix this important typo :-) https://github.com/python/cpython/pull/29798/files

cc @hroncok

@@ -0,0 +1,4 @@
``make regen-all`` now produces the same output when run from a directory
other than the source tree: when building Python out of the source tree.
pegen now strips directory of the "generated by pygen from <FILENAME" header
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Unclosed <

@vstinner vstinner merged commit 93a540d into python:3.9 Nov 26, 2021
@vstinner vstinner deleted the pegen_basename39 branch November 26, 2021 16:24
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.

5 participants