Add PR action (#5)

* Add PR actions * Skip test on windows
HDFGroup · Jul 16, 2020 · 465194b · 465194b
1 parent 806f5b5
commit 465194b
Show file tree

Hide file tree

Showing 4 changed files with 7 additions and 5 deletions.
diff --git a/.github/workflows/main.yml b/.github/workflows/main.yml
@@ -4,8 +4,8 @@ name: hdf5 dev CI
 on:
   push:
     branches: [ develop, hdf5_1_12, hdf5_1_10, hdf5_1_8 ]
-#  pull_request:
-#    branches: [ develop ]
+  pull_request:
+    branches: [ develop, hdf5_1_12, hdf5_1_10, hdf5_1_8 ]
 
 # A workflow run is made up of one or more jobs that can run sequentially or in parallel
 jobs:

diff --git a/config/cmake/HDF5UseFortran.cmake b/config/cmake/HDF5UseFortran.cmake
@@ -435,7 +435,7 @@ set (${HDF_PREFIX}_H5CONFIG_F_RKIND_SIZEOF "INTEGER, DIMENSION(1:num_rkinds) ::
 ENABLE_LANGUAGE (C)
 
 if (NOT CMAKE_VERSION VERSION_LESS "3.14.0")
-include (CheckCSourceRuns)
+  include (CheckCSourceRuns)
 else ()
 #-----------------------------------------------------------------------------
 # The provided CMake C macros don't provide a general compile/run function

diff --git a/config/cmake_ext_mod/ConfigureChecks.cmake b/config/cmake_ext_mod/ConfigureChecks.cmake
@@ -679,4 +679,3 @@ endif ()
 # the cache value is set in it's config file)
 #
 set (${HDF_PREFIX}_CONVERT_DENORMAL_FLOAT 1)
-
diff --git a/test/CMakeLists.txt b/test/CMakeLists.txt
@@ -274,7 +274,7 @@ set (H5_TESTS
     enc_dec_plist
     enc_dec_plist_cross_platform
     getname
-    vfd
+    #vfd # broken on windows
     ros3
     s3comms
     hdfs
@@ -297,6 +297,9 @@ set (H5_TESTS
     vol
     timer
 )
+if (NOT WIN32)
+  list (APPEND ${H5_TESTS} vfd)
+endif ()
 
 macro (ADD_H5_EXE file)
   add_executable (${file} ${HDF5_TEST_SOURCE_DIR}/${file}.c)
Original file line number	Diff line number	Diff line change
Expand Up		@@ -679,4 +679,3 @@ endif ()
		# the cache value is set in it's config file)
		#
		set (${HDF_PREFIX}_CONVERT_DENORMAL_FLOAT 1)