Fix macOS build configuration

Author: AlexanderViand

.github/workflows/pytest-coverage.yml

@@ -33,28 +33,22 @@ jobs:
       with:
         submodules: recursive
 
-    - name: (MacOs only) Install GNU GCC.
+    - name: (MacOs only) Install build dependencies
       if: matrix.os == 'macos-latest'
       run: |
-        brew install gcc libomp
-
-    - name: (MacOs only) Set CC/CXX environment variables to GNU GCC 
-      if: matrix.os == 'macos-latest'
-      run: |
-
-        # Check GCC installation path. Brew installs GCC in /opt/homebrew/bin on Apple Silicon and /usr/local/bin on Intel.
-        if [[ $(uname -m) = "arm64" ]]; then BREW_GCC_PATH="/opt/homebrew/bin"; else BREW_GCC_PATH="/usr/local/bin"; fi
-        echo "GCC installed at $BREW_GCC_PATH"
-
-        # Set CC/CXX environment variables to GNU GCC
-        echo "CC=$BREW_GCC_PATH/$(ls $BREW_GCC_PATH | grep ^gcc-[0-9] | sort -V -r | head -n 1)" >> $GITHUB_ENV
-        echo "CXX=$BREW_GCC_PATH/$(ls $BREW_GCC_PATH | grep ^g++-[0-9] | sort -V -r | head -n 1)" >> $GITHUB_ENV
-        echo "CC=${{ env.CC }}"
-        echo "CXX=${{ env.CXX }}"
-        
-        # Set MACOSX_DEPLOYMENT_TARGET to avoid version mismatch warnings
+        # Install Homebrew LLVM toolchain with OpenMP support
+        brew install llvm libomp
+        if [[ $(uname -m) == "arm64" ]]; then BREW_PREFIX="/opt/homebrew"; else BREW_PREFIX="/usr/local"; fi
+        echo "PATH=$BREW_PREFIX/opt/llvm/bin:$PATH" >> $GITHUB_ENV
+        echo "CC=$BREW_PREFIX/opt/llvm/bin/clang" >> $GITHUB_ENV
+        echo "CXX=$BREW_PREFIX/opt/llvm/bin/clang++" >> $GITHUB_ENV
+        echo "CPPFLAGS=-I$BREW_PREFIX/opt/libomp/include" >> $GITHUB_ENV
+        echo "CFLAGS=-arch $(uname -m)" >> $GITHUB_ENV
+        echo "CXXFLAGS=-arch $(uname -m)" >> $GITHUB_ENV
+        echo "LDFLAGS=-arch $(uname -m) -L$BREW_PREFIX/opt/libomp/lib" >> $GITHUB_ENV
+        echo "ARCHFLAGS=-arch $(uname -m)" >> $GITHUB_ENV
+        echo "CMAKE_OSX_ARCHITECTURES=$(uname -m)" >> $GITHUB_ENV
         echo "MACOSX_DEPLOYMENT_TARGET=$(sw_vers -productVersion)" >> $GITHUB_ENV
-        echo "MACOSX_DEPLOYMENT_TARGET=${{ env.MACOSX_DEPLOYMENT_TARGET }}"
 
     - name: Set up Python ${{ matrix.python-version }}
       uses: actions/setup-python@v4
@@ -71,8 +65,7 @@ jobs:
 
     - name: Install package verbosely
       run: |
-
-        touch .cov # Create a .cov file to trigger Cython compilation with coverage support 
+        python -c "open('.cov','w').close()" # trigger Cython compilation with coverage support
         ls
         python -m pip install -v -v .
 

setup.py

@@ -400,18 +400,27 @@ def build_shared_lib(self, lib_name, build_info):
         lib_file = f"{get_lib_prefix()}{lib_name}{get_lib_suffix('shared')}"
 
         if platform_system == 'Darwin':
-            build_info['extra_link_args'].append(f"-Wl,-install_name,@loader_path/{lib_file}")
-            self.compiler.linker_so = ['-dynamiclib' if val=='-bundle' else val for val in self.compiler.linker_so]
-        self.compiler.link_shared_object(
-            objects,                     
-            lib_file,
-            output_dir          = self.build_clib,
-            target_lang         = language,
-            libraries           = build_info['libraries'],
-            library_dirs        = build_info['library_dirs'],
-            extra_postargs      = build_info['extra_link_args'],
-            build_temp          = self.build_temp,
-        )
+            lib_path = str(Path(self.build_clib).absolute() / lib_file)
+            cmd = [self.compiler.linker_so[0], '-dynamiclib'] + list(objects)
+            cmd += ['-o', lib_path]
+            for lib_dir in build_info['library_dirs']:
+                cmd.append(f'-L{lib_dir}')
+            for lib in build_info['libraries']:
+                cmd.append(f'-l{lib}')
+            cmd += build_info['extra_link_args']
+            cmd.append(f'-Wl,-install_name,@loader_path/{lib_file}')
+            self.compiler.spawn(cmd)
+        else:
+            self.compiler.link_shared_object(
+                objects,
+                lib_file,
+                output_dir          = self.build_clib,
+                target_lang         = language,
+                libraries           = build_info['libraries'],
+                library_dirs        = build_info['library_dirs'],
+                extra_postargs      = build_info['extra_link_args'],
+                build_temp          = self.build_temp,
+            )
         # Post build
         build_info.update({
             'built_lib_files': [str(Path(self.build_clib).absolute() / lib_file)]