[WIP] bpo-1294959: setup.py uses sys.platlibdir#18917
[WIP] bpo-1294959: setup.py uses sys.platlibdir#18917vstinner wants to merge 1 commit intopython:masterfrom vstinner:setup_lib64
Conversation
setup.py now uses sys.platlibdir to configure the compiler library directories and to locate the termcap library.
|
This is the last part of Fedora downstream patch which was part of the commit 8510f43 (PR #18381): |
|
I have questions.
|
|
@hroncok ran rpmdiff to compare the Fedora with the old downstream patches and with the new upstream commit 8510f43: compare the Python package with and without setup.py changes. https://src.fedoraproject.org/rpms/python39/pull-request/31#comment-39392 In short, there is no difference. Moreover, @hroncok, @mcepl and me don't know the rationale of these setup.py changes. I propose to drop them and wait until someone complains. If someone complains, we would know the rationale and so have a good reason to have these setup.py changes. For readline: Fedora libreadline is already linked to libtinfo and so setup.py doesn't go up to For /usr/local/lib vs /usr/local/lib64: I'm not sure why /usr/local/lib is used in the first place. Fedora installs libraries in /usr, not in /usr/local. Maybe it's there for users who install dependencies without Fedora: by installed them manually in /usr/local. Again, since it's unclear how /usr/local is supposed to be used, I prefer to leave it unchanged. I close this issue. |
setup.py now uses sys.platlibdir to configure the compiler library
directories and to locate the termcap library.
https://bugs.python.org/issue1294959