Skip to content
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

bpo-34366: fix FreeBSD with both system lib and libuuid present #8711

Closed
wants to merge 1 commit into from

Conversation

mgorny
Copy link
Contributor

@mgorny mgorny commented Aug 9, 2018

It is entirely possible for a FreeBSD (or another) system to have both
system uuid.h + -lc and external uuid/uuid.h + -luuid libraries
installed simultaneously. If that is the case, prefer the system
library (to avoid unnecessary dependencies) and make sure that the code
does not attempt to combine both.

https://bugs.python.org/issue34366

It is entirely possible for a FreeBSD (or another) system to have both
system uuid.h + -lc and external uuid/uuid.h + -luuid libraries
installed simultaneously.  If that is the case, prefer the system
library (to avoid unnecessary dependencies) and make sure that the code
does not attempt to combine both.
@serhiy-storchaka
Copy link
Member

setup.py no longer used in CPython since 3.12. Could you please update your PR @mgorny?

It is an old issue, how was it solved in existing distributions? Or your configuration is too uncommon? Do we need to backport this change or only change the main branch?

@mgorny
Copy link
Contributor Author

mgorny commented Dec 26, 2023

To be honest, I don't remember if it affects regular FreeBSD (which I haven't used for quite a while already) or if it was specific to Gentoo/FreeBSD (discontinued in ~2019). In either case, I don't really have an environment handy and I don't really have the energy to set one up to check if it is still useful.

@serhiy-storchaka
Copy link
Member

Then I am closing this PR. In any case thank you for it.

@serhiy-storchaka
Copy link
Member

On other hand, there is a similar issue #85077 with alternative solution #20693.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants