CI: Add build-tests for various compilers on Windows and Ubuntu

.github/workflows/ci.yaml

@@ -0,0 +1,47 @@
+on:
+  - push
+
+jobs:
+  build-linux:
+    name: Use clang and gcc via "Unix Makefiles"
+    runs-on: ubuntu-latest
+    strategy:
+      matrix:
+        compiler: [clang, gcc]
+    steps:
+      - uses: actions/checkout@v4
+      - run: |
+          cmake -Bbuild -DCMAKE_C_COMPILER=${{ matrix.compiler }}
+          cmake --build build
+
+  build-windows-ninja:
+    name: Use clang and clang-cl via Ninja
+    runs-on: windows-latest
+    strategy:
+      matrix:
+        # On Windows, clang only enables the WIN32 flag in CMake whereas
+        # clang-cl also enables MSVC, because it has a cl.exe-like interface.
+        compiler: [clang, clang-cl]
+    steps:
+      - uses: actions/checkout@v4
+      - run: |
+          # Use Ninja because the MSBuild toolchain can only configure
+          # alternative compilers via "toolsets", where only ClangCL is
+          # supported.
+          cmake -Bbuild -GNinja -DCMAKE_C_COMPILER=${{ matrix.compiler }}
+          cmake --build build
+
+  build-windows-msbuild:
+    name: Use MSVC and clang-cl via MSBuild
+    runs-on: windows-latest
+    strategy:
+      matrix:
+        toolset:
+          # The default toolset should be the latest 'v143' when nothing is specified
+          -
+          - -TClangCL
+    steps:
+      - uses: actions/checkout@v4
+      - run: |
+          cmake -Bbuild ${{ matrix.toolset }}
+          cmake --build build

CMakeLists.txt

@@ -73,7 +73,7 @@ target_include_directories(${PROJECT_NAME}
          $<INSTALL_INTERFACE:include>)
 
 target_link_libraries(${PROJECT_NAME}
-  PUBLIC $<$<NOT:$<C_COMPILER_ID:MSVC>>:m>)
+  PUBLIC $<$<NOT:$<BOOL:${WIN32}>>:m>)
 
 set_target_properties(${PROJECT_NAME} PROPERTIES
   SOVERSION ${PROJECT_VERSION_MAJOR}

Makefile

@@ -72,12 +72,11 @@ ifneq ($(shared), not-set)
 endif
 
 define run-config
-mkdir -p $(BUILDDIR)
-cd $(BUILDDIR) && cmake $(CURDIR) $(CONFIG_FLAGS)
+cmake $(CURDIR) -B"$(BUILDDIR)" $(CONFIG_FLAGS)
 endef
 
 all clean install: $(BUILDDIR)
-	make -C $(BUILDDIR) $@
+	cmake --build $(BUILDDIR) $@
 
 uninstall:
 	 xargs rm < $(BUILDDIR)/install_manifest.txt

include/gk_arch.h

@@ -53,7 +53,7 @@
 /*************************************************************************
 * Architecture-specific modifications
 **************************************************************************/
-#ifdef WIN32
+#ifdef _WIN32
 typedef ptrdiff_t ssize_t;
 #endif
 

src/error.c

@@ -174,7 +174,7 @@ void gk_NonLocalExit_Handler(int signum)
 /**************************************************************************/
 char *gk_strerror(int errnum)
 {
-#if defined(WIN32) || defined(__MINGW32__)
+#if defined(_WIN32) || defined(__MINGW32__)
   return strerror(errnum);
 #else 
 #ifndef SUNOS

src/string.c

@@ -483,7 +483,7 @@ char *gk_time2str(time_t time)
 
 
 
-#if !defined(WIN32) && !defined(__MINGW32__)
+#if !defined(_WIN32) && !defined(__MINGW32__)
 /************************************************************************/
 /*! \brief Converts a date/time string into its equivalent time_t value
 

src/timers.c

@@ -39,7 +39,7 @@ double gk_CPUSeconds(void)
 #ifdef __OPENMPXXXX__
   return omp_get_wtime();
 #else
-  #if defined(WIN32) || defined(__MINGW32__)
+  #if defined(_WIN32) || defined(__MINGW32__)
     return((double) clock()/CLOCKS_PER_SEC);
   #else
     struct rusage r;