Skip to content

Conversation

@mwtoews
Copy link

@mwtoews mwtoews commented Dec 15, 2025

This (untested) fix adds /usr/local/lib64 to manylinux images (except i686, which doesn't have this dir).

Fixes #1886

@mwtoews mwtoews force-pushed the ld-library-path-with-usr-local-lib branch from 447658e to e1f335d Compare December 15, 2025 02:21
@mwtoews mwtoews force-pushed the ld-library-path-with-usr-local-lib branch from e1f335d to 5879ee6 Compare December 15, 2025 02:47
@mayeut
Copy link
Member

mayeut commented Dec 20, 2025

Thanks for the contribution,
Given it's untested here and commented as such, does this actually fix #1886 (i.e. tested downstream) ?

@mwtoews
Copy link
Author

mwtoews commented Dec 21, 2025

I have since enabled this downstream in shapely/shapely@414315c which works as expected.

I can't think of anything simple to add to docker/tests. The best "simple" would be to build and install some exampled shared library then test to see if it install and correctly links to a shared library in /usr/local/lib64.

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.

Add standard build directories to LD_LIBRARY_PATH for manylinux_2_28?

2 participants