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

Use HDF5 for testing simplex workflows #10887

Merged
merged 1 commit into from
Sep 22, 2020

Conversation

pcafrica
Copy link
Contributor

@pcafrica pcafrica commented Sep 3, 2020

Enable HDF5 in the simplex workflow.

@pcafrica
Copy link
Contributor Author

pcafrica commented Sep 3, 2020

This PR can be safely merged after #10886, which also fixes the simplex/data_out_write_hdf5_01 test failing here (which indeed was not tested in the CI workflow before this PR).

-D DEAL_II_WITH_HDF5="ON" \
-D HDF5_INCLUDE_DIR="/usr/include/hdf5/openmpi/" \
-D HDF5_LIBRARY="/usr/lib/x86_64-linux-gnu/hdf5/openmpi/libhdf5.so" \
-D HDF5_HL_LIBRARY="/usr/lib/x86_64-linux-gnu/hdf5/openmpi/libhdf5_hl.so" \
Copy link
Member

Choose a reason for hiding this comment

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

It should suffice to simply specify something like CMAKE_PREFIX_PATH="/usr/lib/x86_64-linux-gnu/hdf5/openmpi;/usr/include/hdf5/openmpi"

@tamiko tamiko merged commit 513a109 into dealii:master Sep 22, 2020
@peterrum
Copy link
Member

@tamiko Why have you merged this PR? One test has not passed.

@pcafrica
Copy link
Contributor Author

@peterrum see the comment above, the failing test was fixed in #10886!

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.

None yet

3 participants