Title: Generalize configure check for working Python executable
Type: Stage: resolved
Components: Build Versions: Python 3.5
Status: closed Resolution: out of date
Dependencies: Superseder: configure/Makefile doesn't check if "python" command works, needed to build Objects/
View: 26662
Assigned To: Nosy List: brett.cannon, kushal.das, martin.panter, meador.inge
Priority: low Keywords:

Created on 2014-04-15 17:50 by brett.cannon, last changed 2020-03-18 18:30 by brett.cannon. This issue is now closed.

Messages (6)
msg216346 - (view) Author: Brett Cannon (brett.cannon) * (Python committer) Date: 2014-04-15 17:50 has a check that sets ASDLGEN based on what Python interpreter to use for running various scripts which generate files related to the AST. It probably should be generalized so that there's only one check for any script usage in since there is really no need to check multiple times for the same thing.
msg216353 - (view) Author: Meador Inge (meador.inge) * (Python committer) Date: 2014-04-15 18:08
I am not sure I follow.  In I see one check for Python:

  AC_CHECK_PROGS(PYTHON, python$PACKAGE_VERSION python3 python, not-found)
  if test "$PYTHON" = not-found; then
      ASDLGEN="@echo python: $PYTHON! cannot run \$(srcdir)/Parser/ #"

and a couple of uses for it in

  	  $(MKDIR_P) $(AST_H_DIR)

	  $(MKDIR_P) $(AST_C_DIR)

Are you suggesting that the variable name 'ASDLGEN" be renamed to something more general so that any scripts that need to be run *before* building the interpreter can use it?

I don't see any other AC_CHECK_PROGS checks for Python in

Apologies if I am being dense today :-)
msg216355 - (view) Author: Meador Inge (meador.inge) * (Python committer) Date: 2014-04-15 18:15
Ah, okay, this looks in reference to the opcode generation stuff in issue17861.
msg216356 - (view) Author: Brett Cannon (brett.cannon) * (Python committer) Date: 2014-04-15 18:16
You figured out the reason for the interest; I filed the bug faster than Kushal could commit his code. =)
msg284945 - (view) Author: Martin Panter (martin.panter) * (Python committer) Date: 2017-01-08 00:04
It is still not clear what change you were proposing. Perhaps factor out the common code for ADSLGEN and OPCODEHGEN? If so, that has been done as part of Issue 26662 in 3.5+.
msg364547 - (view) Author: Brett Cannon (brett.cannon) * (Python committer) Date: 2020-03-18 18:28
`make regen-all` has probably taken care of this.
Date User Action Args
2020-03-18 18:30:40brett.cannonsetstatus: open -> closed
resolution: out of date
stage: needs patch -> resolved
2020-03-18 18:28:19brett.cannonsetmessages: + msg364547
2017-01-08 00:04:20martin.pantersetsuperseder: configure/Makefile doesn't check if "python" command works, needed to build Objects/

messages: + msg284945
nosy: + martin.panter
2014-04-15 18:16:35brett.cannonsetmessages: + msg216356
2014-04-15 18:15:32meador.ingesetmessages: + msg216355
2014-04-15 18:08:49meador.ingesetnosy: + meador.inge
messages: + msg216353
2014-04-15 17:50:05brett.cannoncreate