-
-
Notifications
You must be signed in to change notification settings - Fork 32.2k
bpo-34032: Add platlibdir as an option for distinguishing /usr/lib and /usr/lib64 #8068
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 and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The new "platlibdir" variable should be documented in Doc/library/sysconfig.rst:
https://docs.python.org/dev/library/sysconfig.html#installation-paths
You have to add it to the list, and also add a ".. versionchanged:: 3.8 (...)" markup to document the change.
IMHO it's worth it to also add a second NEWS entry for the sysconfig change (new "platlibdir" variable).
@@ -0,0 +1,3 @@ | |||
Build system has been modified to allow /usr/lib64 as well as /usr/lib into | |||
install path as well as to the search path for the Linux distributions which | |||
distinguish between 32bit and 64bit libraries. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Can you mention the new --with-custom-platlibdir= configure option somewhere?
Nitpick: I would write "32 bit and 64 bit libraries" ;-) (or "32-bit and 64-bit libraries")
A Python core developer has requested some changes be made to your pull request before we can consider merging it. If you could please address their requests along with any other requests in other reviews from core developers that would be appreciated. Once you have made the requested changes, please leave a comment on this pull request containing the phrase |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The OpenSUSE patch only changes the "unix_prefix" scheme, but the Fedora patch also changes the 'unix_home' scheme. To be honest, I don't understand well this change.
Hmm, this pull request fails on AppVeyor with:
and with
Hmm, I guess this whole thing should be limited just to Unix (non-Windows, non-Mac) systems, right? |
…d /usr/lib64 Build system has been modified to allow /usr/lib64 as well as /usr/lib into install path as well as to the search path for the Linux distributions which distinguish between 32bit and 64bit libraries.
Was this superseded by #11755 ? |
Build system has been modified to allow /usr/lib64 as well as /usr/lib into install path as well as to the search path for the Linux distributions which distinguish between 32bit and 64bit libraries.
https://bugs.python.org/issue34032
@vstinner , @doko42 , who else?