Skip to content

Commit

Permalink
Merge pull request #1743 from NCAR/ejh_more_tests_2
Browse files Browse the repository at this point in the history
Adding more github testing
  • Loading branch information
edwardhartnett authored Aug 31, 2020
2 parents 63662f5 + 261d8b6 commit c569627
Show file tree
Hide file tree
Showing 4 changed files with 15 additions and 13 deletions.
14 changes: 14 additions & 0 deletions .github/workflows/a4.yml
Original file line number Diff line number Diff line change
Expand Up @@ -128,4 +128,18 @@ jobs:
autoreconf -i
./configure --enable-fortran
make -j check
- name: cmake build
run: |
set -x
echo 'export PATH=/home/runner/mpich/bin:$PATH' > .bashrc
source .bashrc
export CC=mpicc
mkdir build
cd build
export LD_LIBRARY_PATH="/home/runner/netcdf-c/lib:/home/runner/pnetcdf/lib:/home/runner/hdf5/lib:/home/runner/mpich/lib:$LD_LIBRARY_PATH"
cmake -Wno-dev -DNetCDF_C_LIBRARY=/home/runner/netcdf-c/lib/libnetcdf.so -DNetCDF_C_INCLUDE_DIR=/home/runner/netcdf-c/include -DPnetCDF_PATH='/home/runner/pnetcdf' -DPIO_ENABLE_FORTRAN=Off -DPIO_HDF5_LOGGING=On -DPIO_USE_MALLOC=On -DPIO_ENABLE_LOGGING=On -DPIO_ENABLE_TIMING=Off .. || (cat CMakeFiles/CMakeOutput.log && cat CMakeFiles/CMakeError.log)
cat config.h
make VERBOSE=1
make tests VERBOSE=1
ctest -VV
6 changes: 0 additions & 6 deletions src/clib/CMakeLists.txt
Original file line number Diff line number Diff line change
Expand Up @@ -112,12 +112,6 @@ if (PnetCDF_C_FOUND)

# Check library for varn functions
set (CMAKE_REQUIRED_LIBRARIES ${PnetCDF_C_LIBRARY})
# check_function_exists (ncmpi_get_varn PnetCDF_C_HAS_VARN)
# if (PnetCDF_C_HAS_VARN)
# target_compile_definitions(pioc
# PUBLIC USE_PNETCDF_VARN
# PUBLIC USE_PNETCDF_VARN_ON_READ)
# endif()
endif ()

#===== Add EXTRAs =====
Expand Down
6 changes: 0 additions & 6 deletions src/flib/CMakeLists.txt
Original file line number Diff line number Diff line change
Expand Up @@ -229,12 +229,6 @@ if (PnetCDF_Fortran_FOUND)

# Check library for varn functions
set (CMAKE_REQUIRED_LIBRARIES ${PnetCDF_Fortran_LIBRARY})
check_function_exists (ncmpi_get_varn PnetCDF_Fortran_HAS_VARN)
if (PnetCDF_Fortran_HAS_VARN)
target_compile_definitions(piof
PUBLIC USE_PNETCDF_VARN
PUBLIC USE_PNETCDF_VARN_ON_READ)
endif()
endif ()

#===== Add EXTRAs =====
Expand Down
2 changes: 1 addition & 1 deletion tests/cunit/CMakeLists.txt
Original file line number Diff line number Diff line change
Expand Up @@ -155,7 +155,7 @@ endif ()
if (PIO_VALGRIND_CHECK)
set (DEFAULT_TEST_TIMEOUT 480)
else ()
set (DEFAULT_TEST_TIMEOUT 240)
set (DEFAULT_TEST_TIMEOUT 480)
endif ()

# All tests need a certain number of tasks, but they should be able to
Expand Down

0 comments on commit c569627

Please sign in to comment.