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

Fix pip install static library not found errors for conda #2683

Merged
merged 1 commit into from
Oct 26, 2024

Conversation

shaform
Copy link
Contributor

@shaform shaform commented Oct 5, 2024

Under certain circumstances Py_ENABLE_SHARED would be False but the static library cannot be found. For example, if you use conda environments or if you use specific versions of MacOS #2109, #2270. pip install would fail in such a situation.

In this PR, we adjust the configurations so that the installation script would attempt to use shared library instead of the static one if no static library is found. This fixes pip install for Python 3.10.15 | packaged by conda-forge. It's not clear if this would fix the MacOS problems mentioned in the above issues though, as I am unable to verify those problems.

@shaform shaform changed the title Use dynamic linking for pip install when static library cannot be found Fix pip install static library not found errors for conda Oct 15, 2024
@xrmx xrmx merged commit 89cb161 into unbit:master Oct 26, 2024
27 checks passed
@xrmx
Copy link
Collaborator

xrmx commented Oct 26, 2024

Thanks!

xrmx pushed a commit to xrmx/uwsgi that referenced this pull request Oct 26, 2024
Under certain circumstances Py_ENABLE_SHARED would be False but the static library cannot be found. For example, if you use conda environments or if you use specific versions of MacOS unbit#2109, unbit#2270. pip install would fail in such a situation.

In this PR, we adjust the configurations so that the installation script would attempt to use shared library instead of the static one if no static library is found. This fixes pip install for Python 3.10.15 | packaged by conda-forge. It's not clear if this would fix the MacOS problems mentioned in the above issues though, as I am unable to verify those problems.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants