bpo-30273: Update sysconfig by vstinner · Pull Request #1464 · python/cpython (original) (raw)
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 andprivacy statement. We’ll occasionally send you account related emails.
Already on GitHub?Sign in to your account
Conversation3 Commits1 Checks0 Files changed
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.Learn more about bidirectional Unicode characters
[ Show hidden characters]({{ revealButtonHref }})
The AST_H_DIR variable was removed from Makefile.pre.in by the commit
a5c62a8 (bpo-23404).
AST_H_DIR was hardcoded to "Include", so replace the removed variable
by its content.
Remove also ASDLGEN variable from sysconfig example since this
variable was also removed.
The AST_H_DIR variable was removed from Makefile.pre.in by the commit a5c62a8 (bpo-23404).
AST_H_DIR was hardcoded to "Include", so replace the removed variable by its content.
Remove also ASDLGEN variable from sysconfig example since this variable was also removed.
I validated that it's possible to install coverage in a virtual environment with this fix:
$ git clean -fdx && ./configure --with-pydebug && make && ./python -m venv venv && ./venv/bin/python -m pip install -U coverage
(...
Successfully installed coverage-4.3.4
I read "Successfully installed coverage-4.3.4" on Travis CI: the change fixed the coverage job ;-)
vstinner added a commit that referenced this pull request
Don't rebuild generated files based on file modification time anymore, the action is now explicit. Replace "make touch" with "make regen-all".
Changes:
Remove "make touch", Tools/hg/hgtouch.py and .hgtouch
Add a new "make regen-all" command to rebuild all generated files
Add subcommands to only generate specific files:
- regen-ast: Include/Python-ast.h and Python/Python-ast.c
- regen-grammar: Include/graminit.h and Python/graminit.c
- regen-importlib: Python/importlib_external.h and Python/importlib.h
- regen-opcode: Include/opcode.h
- regen-opcode-targets: Python/opcode_targets.h
- regen-typeslots: Objects/typeslots.inc
Rename PYTHON_FOR_GEN to PYTHON_FOR_REGEN
pgen is now only built by by "make regen-grammar"
Add $(srcdir)/ prefix to paths to source files to handle correctly compilation outside the source directory
Note: $(PYTHON_FOR_REGEN) is no more used nor needed by "make" default target building Python. (cherry picked from commit a5c62a8)
The AST_H_DIR variable was removed from Makefile.pre.in by the commit a5c62a8 (bpo-23404).
AST_H_DIR was hardcoded to "Include", so replace the removed variable by its content.
Remove also ASDLGEN variable from sysconfig example since this variable was also removed. (cherry picked from commit b109a1d)
vstinner added a commit that referenced this pull request
Don't rebuild generated files based on file modification time anymore, the action is now explicit. Replace "make touch" with "make regen-all".
Changes:
Remove "make touch", Tools/hg/hgtouch.py and .hgtouch
Add a new "make regen-all" command to rebuild all generated files
Add subcommands to only generate specific files:
- regen-ast: Include/Python-ast.h and Python/Python-ast.c
- regen-grammar: Include/graminit.h and Python/graminit.c
- regen-importlib: Python/importlib_external.h and Python/importlib.h
- regen-opcode: Include/opcode.h
- regen-opcode-targets: Python/opcode_targets.h
- regen-typeslots: Objects/typeslots.inc
Rename PYTHON_FOR_GEN to PYTHON_FOR_REGEN
pgen is now only built by by "make regen-grammar"
Add $(srcdir)/ prefix to paths to source files to handle correctly compilation outside the source directory
Note: $(PYTHON_FOR_REGEN) is no more used nor needed by "make" default target building Python. (cherry picked from commit a5c62a8)
The AST_H_DIR variable was removed from Makefile.pre.in by the commit a5c62a8 (bpo-23404).
AST_H_DIR was hardcoded to "Include", so replace the removed variable by its content.
Remove also ASDLGEN variable from sysconfig example since this variable was also removed. (cherry picked from commit b109a1d)
(cherry picked from commit 9d02f56)