diff options
author | Victor Stinner <vstinner@python.org> | 2021-11-26 15:38:50 (GMT) |
---|---|---|
committer | GitHub <noreply@github.com> | 2021-11-26 15:38:50 (GMT) |
commit | b6defde2afe656db830d6fedf74ca5f6225f5928 (patch) | |
tree | 08959c6ba471d717b707eba3d5de150b6ac95f4f | |
parent | 3f024e27c29a57dd4f805aa2431d713ed0fe57b2 (diff) | |
download | cpython-b6defde2afe656db830d6fedf74ca5f6225f5928.zip cpython-b6defde2afe656db830d6fedf74ca5f6225f5928.tar.gz cpython-b6defde2afe656db830d6fedf74ca5f6225f5928.tar.bz2 |
bpo-45866: pegen strips directory of "generated from" header (GH-29777) (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 253b7a0a9fef1d72a4cb87b837885576e68e917c)
-rw-r--r-- | Misc/NEWS.d/next/Build/2021-11-25-13-53-36.bpo-45866.ZH1W8N.rst | 4 | ||||
-rw-r--r-- | Parser/parser.c | 2 | ||||
-rw-r--r-- | Tools/peg_generator/pegen/c_generator.py | 4 | ||||
-rw-r--r-- | Tools/peg_generator/pegen/grammar_parser.py | 2 | ||||
-rw-r--r-- | Tools/peg_generator/pegen/python_generator.py | 4 |
5 files changed, 12 insertions, 4 deletions
diff --git a/Misc/NEWS.d/next/Build/2021-11-25-13-53-36.bpo-45866.ZH1W8N.rst b/Misc/NEWS.d/next/Build/2021-11-25-13-53-36.bpo-45866.ZH1W8N.rst new file mode 100644 index 0000000..efa046d --- /dev/null +++ b/Misc/NEWS.d/next/Build/2021-11-25-13-53-36.bpo-45866.ZH1W8N.rst @@ -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 +Patch by Victor Stinner. diff --git a/Parser/parser.c b/Parser/parser.c index 0b6450c..67515ec 100644 --- a/Parser/parser.c +++ b/Parser/parser.c @@ -1,4 +1,4 @@ -// @generated by pegen.py from ./Grammar/python.gram +// @generated by pegen.py from python.gram #include "pegen.h" #if defined(Py_DEBUG) && defined(Py_BUILD_CORE) diff --git a/Tools/peg_generator/pegen/c_generator.py b/Tools/peg_generator/pegen/c_generator.py index 3079edf..3c3baef 100644 --- a/Tools/peg_generator/pegen/c_generator.py +++ b/Tools/peg_generator/pegen/c_generator.py @@ -1,5 +1,6 @@ import ast from dataclasses import field, dataclass +import os.path import re from typing import Any, Dict, IO, Optional, List, Text, Tuple, Set from enum import Enum @@ -405,7 +406,8 @@ class CParserGenerator(ParserGenerator, GrammarVisitor): def generate(self, filename: str) -> None: self.collect_todo() - self.print(f"// @generated by pegen.py from {filename}") + basename = os.path.basename(filename) + self.print(f"// @generated by pegen.py from {basename}") header = self.grammar.metas.get("header", EXTENSION_PREFIX) if header: self.print(header.rstrip("\n")) diff --git a/Tools/peg_generator/pegen/grammar_parser.py b/Tools/peg_generator/pegen/grammar_parser.py index 70fa5b0..892df5c 100644 --- a/Tools/peg_generator/pegen/grammar_parser.py +++ b/Tools/peg_generator/pegen/grammar_parser.py @@ -1,5 +1,5 @@ #!/usr/bin/env python3.8 -# @generated by pegen from ./Tools/peg_generator/pegen/metagrammar.gram +# @generated by pegen from metagrammar.gram import ast from typing import Optional, Any diff --git a/Tools/peg_generator/pegen/python_generator.py b/Tools/peg_generator/pegen/python_generator.py index b500e3e..4b270aa 100644 --- a/Tools/peg_generator/pegen/python_generator.py +++ b/Tools/peg_generator/pegen/python_generator.py @@ -1,3 +1,4 @@ +import os.path import token from typing import Any, Dict, Optional, IO, Text, Tuple @@ -139,7 +140,8 @@ class PythonParserGenerator(ParserGenerator, GrammarVisitor): def generate(self, filename: str) -> None: header = self.grammar.metas.get("header", MODULE_PREFIX) if header is not None: - self.print(header.rstrip("\n").format(filename=filename)) + basename = os.path.basename(filename) + self.print(header.rstrip("\n").format(filename=basename)) subheader = self.grammar.metas.get("subheader", "") if subheader: self.print(subheader.format(filename=filename)) |