Uh oh!
There was an error while loading.Please reload this page.
- Notifications
You must be signed in to change notification settings - Fork32.2k
gh-128321: SetLIBS
instead ofLDFLAGS
during sqlite3 library build checks#128322
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 ourterms of service andprivacy statement. We’ll occasionally send you account related emails.
Already on GitHub?Sign in to your account
Uh oh!
There was an error while loading.Please reload this page.
Conversation
Uh oh!
There was an error while loading.Please reload this page.
Uh oh!
There was an error while loading.Please reload this page.
LIBS
during sqlite3 library build checksLIBS
instead ofLDFLAGS
during sqlite3 library build checks81376fe
intopython:mainUh oh!
There was an error while loading.Please reload this page.
Thanks for the review! |
This comment was marked as outdated.
This comment was marked as outdated.
This comment was marked as outdated.
This comment was marked as outdated.
…quirements (pythonGH-128322)(cherry picked from commit81376fe)Co-authored-by: Zanie Blue <contact@zanie.dev>Co-authored-by: Erlend E. Aasland <erlend@python.org>
…quirements (pythonGH-128322)(cherry picked from commit81376fe)Co-authored-by: Zanie Blue <contact@zanie.dev>Co-authored-by: Erlend E. Aasland <erlend@python.org>
GH-128355 is a backport of this pull request to the3.12 branch. |
GH-128356 is a backport of this pull request to the3.13 branch. |
…quirements (python#128322)Co-authored-by: Erlend E. Aasland <erlend@python.org>
Uh oh!
There was an error while loading.Please reload this page.
Closes#128321
See the issue for context.
There are a few alternative approaches we could take, such as:
-lm
during checks (which would over-link)PKG_CHECK_MODULES_STATIC
instead ofPKG_CHECK_MODULES
when shared libraries are disabledI'm not really sure of the trade-offs. This seems like the simplest approach to unblock users who can set
PKG_CONFIG
to include the--static
flag as mentioned inhttps://bugs.freedesktop.org/show_bug.cgi?id=19541#c7sqlite3
fails checks due to missing-lm
#128321