This issue tracker has been migrated to GitHub, and is currently read-only.
For more information, see the GitHub FAQs in the Python's Developer Guide.

classification
Title: Makefile missing '/' for some path names
Type: behavior Stage: backport needed
Components: Installation, macOS Versions: Python 3.10, Python 3.9
process
Status: open Resolution:
Dependencies: Superseder:
Assigned To: Nosy List: gwolski, ned.deily, ronaldoussoren, zach.ware
Priority: normal Keywords:

Created on 2022-01-11 23:32 by gwolski, last changed 2022-04-11 14:59 by admin.

Messages (2)
msg410356 - (view) Author: Guntram Wolski (gwolski) Date: 2022-01-11 23:32
In reviewing the output of make -n install, I notice certain paths are missing '/' separators: 

if test "x" != "x" ; then \
	rm -f /usr/local/python3.9.9/binpython3.9-32; \
	lipo  \
		-output /usr/local/python3.9.9/bin/python3.9-32 \
		/usr/local/python3.9.9/bin/python3.9; \
fi

There should be a '/' between bin and python3.9-32 and other lines have the same issue. The Makefile source lines where I see this error are:

        if test "x$(LIPO_32BIT_FLAGS)" != "x" ; then \
-->                rm -f $(DESTDIR)$(BINDIR)python$(VERSION)-32$(EXE); \
                lipo $(LIPO_32BIT_FLAGS) \
                        -output $(DESTDIR)$(BINDIR)/python$(VERSION)-32$(EXE) \
                        $(DESTDIR)$(BINDIR)/python$(VERSION)$(EXE); \
        fi
        if test "x$(LIPO_INTEL64_FLAGS)" != "x" ; then \
-->                rm -f $(DESTDIR)$(BINDIR)python$(VERSION)-intel64$(EXE); \
                lipo $(LIPO_INTEL64_FLAGS) \
                        -output $(DESTDIR)$(BINDIR)/python$(VERSION)-intel64$(EXE) \
                        $(DESTDIR)$(BINDIR)/python$(VERSION)$(EXE); \
        fi


I think the lines I've marked with --> above should have the '/' after $(BINDIR).... ????
msg410413 - (view) Author: Zachary Ware (zach.ware) * (Python committer) Date: 2022-01-12 16:45
This has been fixed in 3.11 with GH-28659, but the fix hasn't been backported due to the unlikelihood of this issue actually being a problem and a procedural wrinkle in the original PR.  I suspect a clean backport would be accepted, but this is still a rather low-priority issue.
History
Date User Action Args
2022-04-11 14:59:54adminsetgithub: 90509
2022-01-12 16:45:39zach.waresettype: behavior
components: + macOS
versions: + Python 3.10
nosy: + ned.deily, zach.ware, ronaldoussoren

messages: + msg410413
stage: backport needed
2022-01-11 23:32:47gwolskicreate