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

Upgrade to hdf5 1.8.18 #6

Merged
merged 1 commit into from
Jun 8, 2017
Merged

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.

@leouieda
Copy link
Member

leouieda commented Jun 2, 2017

@qwhelan thanks for the PR! There seems to be a conflict between hdf5 and cmake, though I can't see a reason why. There don't seem to be any clashing dependencies.

@@ -23,7 +23,7 @@ requirements:
- gdal 2.1.*
- ghostscript
- libnetcdf 4.4.*
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It is not cmake, sometimes conda gets "confused" resolving package conflicts, the issue lies with libnetcdf and gdal. Both dependencies must be available with latest hdf5 before we re-triggerthe CIs here.

Copy link
Contributor Author

@qwhelan qwhelan Jun 2, 2017

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Yeah, I need to fix libnetcdf to fix gdal too. Is there a way for me to re-trigger the CIs without amending the commit?

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Is there a way for me to re-trigger the CIs without amending the commit?

Closing and re-opening usually does the trick. If not just ping me and I will restart the CIs.

@qwhelan qwhelan closed this Jun 7, 2017
@qwhelan qwhelan reopened this Jun 7, 2017
@ocefpaf ocefpaf merged commit ef9a87d into conda-forge:master Jun 8, 2017
@ocefpaf
Copy link
Member

ocefpaf commented Jun 8, 2017

Thanks for another one @qwhelan!

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.

4 participants