Title: Make fails Objects/
Type: compile error Stage:
Components: Build, Interpreter Core Versions: Python 3.4, Python 3.5
Status: closed Resolution: duplicate
Dependencies: Superseder:
Assigned To: Nosy List: martin.panter, masamoto
Priority: normal Keywords: patch

Created on 2015-04-23 04:47 by masamoto, last changed 2016-07-22 03:41 by martin.panter. This issue is now closed.

File name Uploaded Description Edit masamoto, 2015-04-23 04:50 review
Messages (3)
msg241843 - (view) Author: Masayuki Yamamoto (masamoto) * Date: 2015-04-23 04:47
Objects/ is broken, and make stop on cygwin-1.7.35-i686.

If built-in python is not found in make processing, Objects/ generating will fail.
Because Objects/ is generated using $(PYTHON) in Makefile, but  $(PYTHON) variable is a hard-coding value that isn't check a built-in python existent.

make steps to fail and error message:
$ ./configure --prefix=/opt/py34
$ touch Include/typeslots.h
$ make
python.exe ./Objects/ < ./Include/typeslots.h > Objects/
/bin/sh: python.exe: command not found
Makefile:851: recipe for target 'Objects/' failed
make: *** [Objects/] Error 127

Cygwin python executable files:
$ find /usr/bin/ -name 'python*.exe'
I wrote the patch to replace $(PYTHON).

Add TYPESLOTSGEN to  TYPESLOTSGEN values is the built-in python.
If the built-in python is not found, TYPESLOTSGEN echoes message that is skipping like ASDLGEN.
Generated the configure script using autoreconf.
Modify to replace $(PYTHON) to $(TYPESLOTSGEN).
And remove $(PYTHON) that isn't used anywhere.  It seems already needless.

Although I understand that the generated file touch can avoid to fail regenerate, I want to check built-in python to generate safer the Objects/
msg270969 - (view) Author: Martin Panter (martin.panter) * (Python committer) Date: 2016-07-22 03:41
Sorry nobody acted on your patch earlier. This is the same as Issue 26662, which has a more general patch that does the same sort of thing.
Date User Action Args
2016-07-22 03:41:40martin.pantersetstatus: open -> closed

nosy: + martin.panter
messages: + msg270969

resolution: duplicate
2015-04-23 04:50:39masamotosetfiles: +
keywords: + patch
messages: + msg241844
2015-04-23 04:47:10masamotocreate