BUG: Workaround for Elastix macOS specific OpenMP CI issue

xref SuperElastix/elastix#650
N-Dekker · Aug 23, 2022 · 82255da · 82255da
1 parent e71c7d5
commit 82255da
Showing 1 changed file with 17 additions and 5 deletions.
diff --git a/.github/workflows/build-test-package.yml b/.github/workflows/build-test-package.yml
@@ -108,10 +108,21 @@ jobs:
         if(WIN32)
           set(ENV{PATH} "\${CTEST_DASHBOARD_ROOT}/ITK-build/bin;\$ENV{PATH}")
         endif()
-        set(dashboard_cache "
-        ITK_DIR:PATH=\${CTEST_DASHBOARD_ROOT}/ITK-build
-        BUILD_TESTING:BOOL=ON
-        ")
+        if(APPLE)
+          # Switch off OpenMP, to work around macOS specific CI errors, saying:
+          # > fatal error: ‘omp.h’ file not found
+          # See https://github.com/SuperElastix/elastix/commit/72c8590cd563e3f72a2dfbbc69ad2d05a6a3b936
+          set(dashboard_cache "
+          ITK_DIR:PATH=\${CTEST_DASHBOARD_ROOT}/ITK-build
+          BUILD_TESTING:BOOL=ON
+          ELASTIX_USE_OPENMP:BOOL=OFF
+          ")
+        else()
+          set(dashboard_cache "
+          ITK_DIR:PATH=\${CTEST_DASHBOARD_ROOT}/ITK-build
+          BUILD_TESTING:BOOL=ON
+          ")
+        endif()
         string(TIMESTAMP build_date "%Y-%m-%d")
         message("CDash Build Identifier: \${build_date} \${CTEST_BUILD_NAME}")
         message("CTEST_SITE = \${CTEST_SITE}")
@@ -190,7 +201,8 @@ jobs:
       run: |
         export ITK_PACKAGE_VERSION=${{ env.itk-wheel-tag }}
         export MACOSX_DEPLOYMENT_TARGET=10.9
-        ./macpython-download-cache-and-build-module-wheels.sh
+        # See https://github.com/SuperElastix/elastix/commit/72c8590cd563e3f72a2dfbbc69ad2d05a6a3b936 for OpenMP workaround
+        ./macpython-download-cache-and-build-module-wheels.sh --cmake_options "-DELASTIX_USE_OPENMP:BOOL=OFF"
 
     - name: Publish Python package as GitHub Artifact
       uses: actions/upload-artifact@v1