Skip to content

Upgrade to hdf5 1.8.18#21

Merged
ocefpaf merged 2 commits intoconda-forge:masterfrom
qwhelan:hdf5_1.8.18
Jun 6, 2017
Merged

Upgrade to hdf5 1.8.18#21
ocefpaf merged 2 commits intoconda-forge:masterfrom
qwhelan:hdf5_1.8.18

Conversation

@qwhelan
Copy link
Contributor

@qwhelan qwhelan commented Jun 2, 2017

A few CVEs were reported and fixed in hdf5 back in November 2016: http://blog.talosintelligence.com/2016/11/hdf5-vulns.html

The hdf5 release notes for that release are available here: https://support.hdfgroup.org/ftp/HDF5/current18/src/hdf5-1.8.18-RELEASE.txt

For additional info, please see: conda-forge/hdf5-feedstock#68 and conda-forge/hdf5-feedstock#71

@conda-forge-linter
Copy link

Hi! This is the friendly automated conda-forge-linting service.

I just wanted to let you know that I linted all conda-recipes in your PR (recipe) and found it was in an excellent condition.

@ocefpaf
Copy link
Member

ocefpaf commented Jun 2, 2017

See #20 (we actually have a Windows problem there BTW.)

@qwhelan
Copy link
Contributor Author

qwhelan commented Jun 6, 2017

@ocefpaf I've resolved the Windows issue. The hdf5 feedstock added the --enable-threadsafe flag in conda-forge/hdf5-feedstock#48, which I believe is the cause of the issue we saw here.

Patching CMakeLists.txt to load the Threads library on Windows appears to be sufficient to resolve the issue.

@ocefpaf
Copy link
Member

ocefpaf commented Jun 6, 2017

Awesome! Thanks @qwhelan!!

@ocefpaf ocefpaf merged commit 2b9948b into conda-forge:master Jun 6, 2017
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.

3 participants