Skip to content

Add multiple hdf5 versions for static building - #207

Open
mulimoen wants to merge 6 commits into
aldanor:masterfrom
mulimoen:feature/static-multi-versions
Open

Add multiple hdf5 versions for static building#207
mulimoen wants to merge 6 commits into
aldanor:masterfrom
mulimoen:feature/static-multi-versions

Conversation

@mulimoen

@mulimoen mulimoen commented Mar 23, 2022

Copy link
Copy Markdown
Collaborator

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

mulimoen commented Apr 6, 2022

Copy link
Copy Markdown
Collaborator Author

@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
Copy Markdown
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
Copy Markdown
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

aldanor commented Jun 9, 2023

Copy link
Copy Markdown
Owner

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

@mulimoen

mulimoen commented Jun 9, 2023

Copy link
Copy Markdown
Collaborator Author

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

@aldanor

aldanor commented Jun 9, 2023

Copy link
Copy Markdown
Owner

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.

2 participants