Skip to content
Snippets Groups Projects
Commit 88f261e9 authored by Atkins, Charles Vernon's avatar Atkins, Charles Vernon Committed by GitHub
Browse files

Merge pull request #100 from chuckatkins/fix-hdf5-include-paths

Use the HDF5_C_INCLUDE_DIRS instead of HDF5_INCLUDE_DIRS
parents a4b56082 ac72aecd
No related branches found
No related tags found
No related merge requests found
......@@ -46,7 +46,7 @@ if(ADIOS_USE_MPI)
IO_hdf5_a.cpp
)
target_include_directories(heatTransfer_write_hdf5
PRIVATE ${MPI_C_INCLUDE_PATH} ${HDF5_INCLUDE_DIRS}
PRIVATE ${MPI_C_INCLUDE_PATH} ${HDF5_C_INCLUDE_DIRS}
)
target_link_libraries(heatTransfer_write_hdf5
${MPI_C_LIBRARIES} ${HDF5_C_LIBRARIES}
......
......@@ -102,8 +102,7 @@ if(ADIOS_USE_HDF5)
"of ADIOS is being built, which requires a sequential HDF5."
)
endif()
target_include_directories(adios2 PRIVATE ${HDF5_INCLUDE_DIRS})
target_include_directories(adios2 PRIVATE ${HDF5_C_INCLUDE_DIRS})
target_sources(adios2 PRIVATE
engine/hdf5/HDF5ReaderP.cpp
engine/hdf5/HDF5WriterP.cpp
......
......@@ -3,6 +3,10 @@
# accompanying file Copyright.txt for details.
#------------------------------------------------------------------------------#
find_package(HDF5 REQUIRED)
add_executable(TestHDF5WriteRead TestHDF5WriteRead.cpp)
target_link_libraries(TestHDF5WriteRead adios2 gtest)
target_include_directories(TestHDF5WriteRead PRIVATE ${HDF5_C_INCLUDE_DIRS})
target_link_libraries(TestHDF5WriteRead adios2 gtest ${HDF5_C_LIBRARIES})
add_test(NAME adios2::engine::hdf5::write_read COMMAND TestHDF5WriteRead)
0% Loading or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment