Fix for cmake and new workflow strategy

Author: vepadulano

.github/workflows/python_wheel_build.yml

@@ -14,68 +14,65 @@ on:
     - cron: '01 1 * * *'
 
 jobs:
-  python_wheel:
-    runs-on:
-      - self-hosted
-      - linux
-      - x64
-    env:
-      WORKDIR: /home/runner/work/root
-    container:
-      image: registry.cern.ch/root-ci/ubuntu2404:buildready
-      options: '--security-opt label=disable --user="ubuntu" --rm'
+  cp38-manylinux_x86_64:
+    runs-on: ubuntu-latest
     steps:
       - name: Checkout
         uses: actions/checkout@v4
         with:
           ref: ${{ inputs.branch }}
 
-      - name: Setup Python
-        uses: actions/setup-python@v5
+      # - name: Build wheel
+      #   uses: pypa/[email protected]
+      #   env:
+      #     CIBW_BUILD: cp38-manylinux_x86_64
+      #   with:
+      #     output-dir: ${{ env.WORKDIR }}/wheelhouse
+      
+      - name: Mock step
+        run: |
+          mkdir -p wheelhouse
+          touch wheelhouse/wheel1.whl
+          touch wheelhouse/wheel2.whl
+
+      - name: Upload wheel
+        uses: actions/upload-artifact@v4
         with:
-          python-version: 3.13
+          name: cp38-manylinux_x86_64
+          path: wheelhouse/*
 
-      # - name: Set up Python Virtual Environment
-      #   run: |
-      #     python -m venv /py-venv/cibuildwheel-env
-      #     . /py-venv/cibuildwheel-env/bin/activate
-      #     echo PATH=$PATH >> $GITHUB_ENV
+  create_and_upload_wheel_registry:
+      needs: cp38-manylinux_x86_64
+      runs-on: ubuntu-latest
+      steps:
 
-      - name: Update repositories
-        run: apt-get update
+      - name: Download artifact
+        uses: actions/download-artifact@v4
+        with:
+          name: cp38-manylinux_x86_64
+
+      - name: Setup Python
+        uses: actions/setup-python@v5
 
       - name: Install Kerberos utilities
-        run: apt-get install -y krb5-user
+        run: sudo apt-get install -y krb5-user
 
       - name: Install XRootD client
-        run: apt-get -y install xrootd-client
-
-      - name: Configure large ccache
-        if: ${{ matrix.is_special }}
-        run: |
-          ccache -o max_size=5G
-          ccache -p || true
-          ccache -s || true
-
-      - name: Build wheels
-        env: 
-          PIP_ROOT_USER_ACTION: ignore
-          PIP_DISABLE_PIP_VERSION_CHECK: 1
-        uses: pypa/[email protected]
-        with:
-          output-dir: ${{ env.WORKDIR }}/wheelhouse
+        run: sudo apt-get -y install xrootd-client
 
       - name: Create PEP503-compliant wheel registry
         run: |
+          mkdir -p wheelhouse
+          mv *.whl wheelhouse
           python -m pip install --upgrade pip
           python -m pip install simple503
-          simple503 ${{ env.WORKDIR }}/wheelhouse
+          simple503 wheelhouse
 
       - name: Upload wheel registry to github
         uses: actions/upload-artifact@v4
         with:
-          name: cibw-wheels-test
-          path: ${{ env.WORKDIR }}/wheelhouse/*
+          name: cibw-wheels-registry
+          path: wheelhouse
 
       - name: Upload wheel to EOS
         env:

bindings/pyroot/cppyy/cppyy-backend/CMakeLists.txt

@@ -31,6 +31,14 @@ if(MSVC)
 endif()
 # Set the suffix to '.so' and the prefix to 'lib'
 set_target_properties(${libname} PROPERTIES  ${ROOT_LIBRARY_PROPERTIES})
+
+target_link_libraries(${libname} Core ${CMAKE_DL_LIBS})
+
+# cppyy uses ROOT headers from binary directory
+add_dependencies(${libname} move_headers)
+
+set_property(GLOBAL APPEND PROPERTY ROOT_EXPORTED_TARGETS ${libname})
+
 # It is crucial to add the path to the other ROOT libraries for compatibility with pip build backends.
 # After the `pip install` step, the target install user directory will look like:
 # site-packages/
@@ -42,14 +50,6 @@ set_target_properties(${libname} PROPERTIES  ${ROOT_LIBRARY_PROPERTIES})
 # -- ROOT/
 # With most of the ROOT libraries being stored in `ROOT/lib`. The RPATH must then be updated for the CPython extension
 # shared libraries.
-set_target_properties(${libname} PROPERTIES  INSTALL_RPATH "${CMAKE_INSTALL_RPATH}:$ORIGIN/ROOT/lib")
-target_link_libraries(${libname} Core ${CMAKE_DL_LIBS})
-
-# cppyy uses ROOT headers from binary directory
-add_dependencies(${libname} move_headers)
-
-set_property(GLOBAL APPEND PROPERTY ROOT_EXPORTED_TARGETS ${libname})
-
 if(NOT MSVC)
   # Make sure that relative RUNPATH to main ROOT libraries is always correct.
 
@@ -64,7 +64,8 @@ if(NOT MSVC)
   else()
     set_target_properties(${libname} PROPERTIES
         BUILD_RPATH "$ORIGIN/${pymoduledir_to_libdir_build}"
-        INSTALL_RPATH "$ORIGIN/${pymoduledir_to_libdir_install}"
+        # ATTENTION: CRUCIAL! See comment above
+        INSTALL_RPATH "$ORIGIN/${pymoduledir_to_libdir_install}:${CMAKE_INSTALL_RPATH}:$ORIGIN/ROOT/lib"
     )
   endif()
 

pyproject.toml

@@ -22,7 +22,6 @@ root = "ROOT._rootcli:main"
 [tool.cibuildwheel]
 # Increase pip debugging output
 build-verbosity = 1
-build="cp3{8,9,10,11,12,13}-manylinux_x86_64"
 manylinux-x86_64-image = "manylinux_2_28"
 
 # Install system libraries