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

Old version of HDF5 in wheels #115

Open
mgeplf opened this issue Jan 6, 2021 · 4 comments
Open

Old version of HDF5 in wheels #115

mgeplf opened this issue Jan 6, 2021 · 4 comments
Assignees

Comments

@mgeplf
Copy link
Contributor

mgeplf commented Jan 6, 2021

This commit uses the hdf5-devel: 9aba1ea

This is a really old version of hdf5 (1.8.12? circa 2013). Previously we were building a much newer version (hdf5-1.10.5, from 2019?)

This should be updated.

@pramodk
Copy link

pramodk commented Jan 6, 2021

I am curious about shipping hdf5 version under wheel and interoperability aspects. What if another python package is loaded first with different/older HDF5 version? Does everything just works?

@mgeplf
Copy link
Contributor Author

mgeplf commented Jan 6, 2021

What if another python package is loaded first with different/older HDF5 version?

They are rpath'd in, but subject to the normal dynamic linking rules, so basically there isn't a lot of control - this is the main reason why having a somewhat modern version is preferable, imo.

In one of the other projects, I was compiling hdf5 statically into the wheel, which is prob. the best way, but I don't know how easy that is with these new-fangled wheel building techniques.

Does everything just works?

HDF5 does a good job of compat/A(B|P)I stability (at least compared to most), so it's slightly less of a problem, but still something that can bite someone, sadly.

@tomdele
Copy link
Contributor

tomdele commented Jun 30, 2021

Not sure about the status of this one... I will reassign to you @mgeplf so you can dispatch if needed or close if this is fine.

@tomdele tomdele assigned mgeplf and unassigned tomdele Jun 30, 2021
@alkino
Copy link
Member

alkino commented Nov 26, 2021

For manylinux2014, we still depends of hdf5 : 1.8.12

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

No branches or pull requests

4 participants