Fix building Python bindings with Clang and add corresponding CI actions (#320)

Author: henrij22

.github/workflows/debian.yml

@@ -25,15 +25,13 @@ jobs:
   Build:
     name: ${{ matrix.config.name }}
     runs-on: ubuntu-latest
-    container:
-      image: ikarusproject/ikarus-dev:latest # this does not work for the python binding tests since the dune base bindings are configured with gcc and here ikarus her executed with clang, we may use ikarus-dev-clang later here
-      options: --memory-swap="20g" --memory="20g" --cpus="2" --user root
     strategy:
       fail-fast: false
       matrix:
         config:
           - {
               name: "Clang-16-Debug",
+              container: "ikarusproject/ikarus-dev-clang:latest",
               config: Debug,
               compilerC: clang-16,
               compilerCxx: clang++-16,
@@ -42,14 +40,34 @@ jobs:
             }
           - {
               name: "Clang-16-Release",
+              container: "ikarusproject/ikarus-dev-clang:latest",
               config: Release,
               compilerC: clang-16,
               compilerCxx: clang++-16,
               lib: "-stdlib=libc++",
               testRegex: "cpp",
             }
+          - {
+              name: "Clang-16-Python-Release",
+              container: "ikarusproject/ikarus-dev-clang:latest",
+              config: Release,
+              compilerC: clang-16,
+              compilerCxx: clang++-16,
+              lib: "-stdlib=libc++",
+              testRegex: "python",
+            }
+          - {
+              name: "Clang-16-Python-Debug",
+              container: "ikarusproject/ikarus-dev-clang:latest",
+              config: Debug,
+              compilerC: clang-16,
+              compilerCxx: clang++-16,
+              lib: "-stdlib=libc++",
+              testRegex: "python",
+            }
           - {
               name: "Gcc-12-Debug",
+              container: "ikarusproject/ikarus-dev-gcc:latest",
               config: Debug,
               compilerC: gcc-12,
               compilerCxx: g++-12,
@@ -58,20 +76,36 @@ jobs:
             }
           - {
               name: "Gcc-12-Release",
+              container: "ikarusproject/ikarus-dev-gcc:latest",
               config: Release,
               compilerC: gcc-12,
               compilerCxx: g++-12,
               lib: "",
               testRegex: "cpp",
             }
           - {
-              name: "Gcc-12-Python",
+              name: "Gcc-12-Python-Release",
+              container: "ikarusproject/ikarus-dev-gcc:latest",
               config: Release,
               compilerC: gcc-12,
               compilerCxx: g++-12,
               lib: "",
               testRegex: "python",
             }
+          - {
+            name: "Gcc-12-Python-Debug",
+            container: "ikarusproject/ikarus-dev-gcc:latest",
+            config: Debug,
+            compilerC: gcc-12,
+            compilerCxx: g++-12,
+            lib: "",
+            testRegex: "python",
+          }
+    container:
+      image: ${{ matrix.config.container }}
+      options: --memory-swap="20g" --memory="20g" --cpus="2" --user root
+    env:
+      IKARUS_PYTHON_TEST_BUILD_TYPE_OVERRIDE: ${{ matrix.config.config }} 
     steps:
       - uses: actions/checkout@v4
         with:

.github/workflows/testpythonpackage.yml

@@ -21,11 +21,14 @@ on:
       - "**.md"
   workflow_dispatch:
 
+env:
+  IKARUS_PYTHON_TEST_BUILD_TYPE_OVERRIDE: "Debug"
+
 jobs:
   build-n-test:
     runs-on: ubuntu-latest
     container:
-      image: ikarusproject/ikarus-dev:latest
+      image: ikarusproject/ikarus-dev-clang:latest
       options: --memory-swap="20g" --memory="20g" --cpus="2" --user root
 
     steps:

ikarus/python/test/debug_info.py

@@ -1,15 +1,26 @@
 # SPDX-FileCopyrightText: 2021-2024 The Ikarus Developers [email protected]
 # SPDX-License-Identifier: LGPL-3.0-or-later
 import os
+import dune.generator as generator
+from logging import info
 
 os.environ["DUNE_LOG_LEVEL"] = "debug"
 os.environ["DUNE_SAVE_BUILD"] = "terminal"
 
 
 def setDebugFlags():
-    import dune.generator as generator
+    def apply_debug_flags():
+        generator.setFlags("-g ", noChecks=False)
+
+    # Check if the environment variable is set
+    build_type = os.environ.get("IKARUS_PYTHON_TEST_BUILD_TYPE_OVERRIDE")
 
-    generator.setFlags("-g ", noChecks=False)
+    # Apply flags only if the build type is Debug, or if the variable is not set
+    if build_type == "Debug" or build_type is None:
+        apply_debug_flags()
+        info("JIT Python Bindings BUILD_TYPE: Debug")
+    else:
+        info("JIT Python Bindings BUILD_TYPE: Release")
 
 
 def unsetDebugFlags():

python/ikarus/CMakeLists.txt

@@ -12,9 +12,10 @@ dune_add_pybind11_module(NAME _ikarus)
 
 set_property(
   TARGET _ikarus
-  PROPERTY LINK_LIBRARIES dunecommon ikarus _io
+  PROPERTY LINK_LIBRARIES dunecommon ikarus
   APPEND
 )
+add_dependencies(_ikarus _io)
 
 if(SKBUILD)
   install(TARGETS _ikarus LIBRARY DESTINATION python/ikarus)