Skip to content

Add HDF package#339

Merged
tgamblin merged 5 commits intospack:developfrom
adamjstewart:features/hdf
Jan 26, 2016
Merged

Add HDF package#339
tgamblin merged 5 commits intospack:developfrom
adamjstewart:features/hdf

Conversation

@adamjstewart
Copy link
Copy Markdown
Member

This package is for HDF4. Currently there is only an HDF5 package. Note that the source code, as well as my package, are named HDF instead of HDF4.

Possible bugs:
spack versions hdf does not find any Remote versions. However, this could be said about a lot of packages that have sites with weird naming conventions. spack fetch hdf@<version> works just fine, so I don't think it's a problem.

* develop: (85 commits)
  Make libxcb compile with gcc 4.9.
  disable parallel install for glib (found races)
  disable cairo-trace, which is incompatible with older libiberty.h.
  Fix LaunchMon on newer gcc versions.
  Simplify output redirection in spack.util.executable
  Allow completely empty config files.
  Update cmake package to use http in all URLs.
  Fix create, diy, edit, and repo commands to use multiple repos.
  Add namespace option to find command.
  Temporary fix: Clang is the default compiler on Mac OS X.
  Netcdf requires cmake >=2.8.12
  Mbedtls depends on cmake
  Update libtool to 2.4.6
  Update Julia to 0.4.3
  Make binutils build on OS X
  Add verbs and psm variants
  py-matplotlib: depend on freetype
  py-numpy: make blas/lapack optional dependencies
  qhull: apply patch to work with libc++'s standard library
  New package GNU tar
  ...
@adamjstewart
Copy link
Copy Markdown
Member Author

Do you think SZip should be a variant?

Some of the configure options I chose are enabled by default. I'm not sure if there's any consensus on whether or not to include options that are already enabled.

@tgamblin
Copy link
Copy Markdown
Member

Making it a variant is generally useful. You can make it default to enabled, and then the user can at least request that it be disabled if they can't build with it for some reason.

@tgamblin
Copy link
Copy Markdown
Member

Looks good! FYI: I don't get notifications when you push commits, even though they show up on the PR. If you want me to get to it faster add a comment. I wish I knew how to add notifications for additional commits.

tgamblin added a commit that referenced this pull request Jan 26, 2016
@tgamblin tgamblin merged commit d7bd023 into spack:develop Jan 26, 2016
@adamjstewart adamjstewart deleted the features/hdf branch January 27, 2016 15:09
matz-e pushed a commit to matz-e/spack that referenced this pull request Apr 27, 2020
climbfuji added a commit to climbfuji/spack that referenced this pull request Oct 6, 2023
Bug fix for py-awscrt on Linux (use spack openssl/libcrypto)
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