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

Add multiple hdf5 versions for static building #207

Open
wants to merge 6 commits into
base: master
Choose a base branch
from

Conversation

mulimoen
Copy link
Collaborator

@mulimoen mulimoen commented Mar 23, 2022

This adds a feature toggle to allow more recent versions of hdf5, 1.12.1 or 1.13.1. Crate size is 9.2M, should still be small enough to be uploaded to crates.io.

@mulimoen mulimoen force-pushed the feature/static-multi-versions branch 3 times, most recently from ca82a36 to 78e1908 Compare March 23, 2022 18:10
@mulimoen mulimoen changed the title WIP: Add multiple hdf5 versions for static building Add multiple hdf5 versions for static building Mar 23, 2022
@mulimoen mulimoen requested a review from aldanor March 23, 2022 18:45
@mulimoen mulimoen force-pushed the feature/static-multi-versions branch 2 times, most recently from b884c57 to 4ac7433 Compare April 6, 2022 20:27
@mulimoen
Copy link
Collaborator Author

mulimoen commented Apr 6, 2022

@aldanor This PR also fixes CI

@mulimoen mulimoen force-pushed the feature/static-multi-versions branch from 9eaa6eb to 571ca1b Compare April 8, 2022 21:28
@mulimoen mulimoen force-pushed the feature/static-multi-versions branch 2 times, most recently from 8a210c8 to daef5cc Compare April 26, 2022 05:44
This was referenced May 3, 2022
@mulimoen
Copy link
Collaborator Author

@aldanor It would be great if you could have a look at this!

@mulimoen mulimoen force-pushed the feature/static-multi-versions branch from 43969c5 to cb42392 Compare October 27, 2022 15:56
@mulimoen
Copy link
Collaborator Author

I replaced hdf5 1.13 (which was a beta lineage) with 1.14.0. This requires #227

@mulimoen mulimoen force-pushed the feature/static-multi-versions branch from 52777e0 to f031764 Compare January 17, 2023 09:28
@aldanor
Copy link
Owner

aldanor commented Jun 9, 2023

Do we want to go ahead with this? If so, 1.13->1.14 and will need a rebase?

@mulimoen
Copy link
Collaborator Author

mulimoen commented Jun 9, 2023

Would be easier to bump the bundled hdf5-c to 1.14 instead of maintaining three versions

@aldanor
Copy link
Owner

aldanor commented Jun 9, 2023

Probably, yea... Now that 1.14 is stable, is out on most platforms and package managers and will be a default version of choice for a long while anyway, I guess, we can just bundle it as a default as well.

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.

None yet

2 participants