Skip to content

Commit

Permalink
Pinning conduit to output files compatible with hdf5@1.8 (#1052)
Browse files Browse the repository at this point in the history
* Pinning conduit to output files compatible with hdf5@1.8

* Adding in #ifdef for H5F_LIBVER_V18

* Fixing whitespace.
  • Loading branch information
rblake-llnl committed Dec 22, 2022
1 parent dc77d7e commit 2e884a6
Showing 1 changed file with 6 additions and 4 deletions.
10 changes: 6 additions & 4 deletions src/libs/relay/conduit_relay_io_hdf5.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -2939,11 +2939,13 @@ create_hdf5_file_access_plist()
// https://forum.hdfgroup.org/t/seconding-the-request-for-h5pset-libver-bounds-1-8-x-file-compat-option/4056
// so only enable H5F_LIBVER_LATEST if we are using hdf5 1.8

if(major_num == 1 && minor_num == 8)
if(major_num == 1 && minor_num >= 8)
{
h5_status = H5Pset_libver_bounds(h5_fa_props,
H5F_LIBVER_LATEST,
H5F_LIBVER_LATEST);
#if H5_VERSION_GE(1, 10, 2)
h5_status = H5Pset_libver_bounds(h5_fa_props, H5F_LIBVER_V18, H5F_LIBVER_V18);
#else
h5_status = H5Pset_libver_bounds(h5_fa_props, H5F_LIBVER_LATEST, H5F_LIBVER_LATEST);
#endif

CONDUIT_CHECK_HDF5_ERROR(h5_status,
"Failed to set libver options for "
Expand Down

0 comments on commit 2e884a6

Please sign in to comment.