Merge pull request #28 from ahoarau/jrl-next

Refactor CMake with JRL CMake Modules v2

Author: ManifoldFR

.gersemirc

@@ -1,4 +1,4 @@
-definitions: [./CMakeLists.txt, ./cmake, ./tests]
-line_length: 80
+definitions: [./CMakeLists.txt, ./tests]
+line_length: 100
 indent: 2
 warn_about_unknown_commands: false

.github/workflows/macos-linux-windows-pixi.yml

@@ -55,8 +55,6 @@ jobs:
 
     steps:
     - uses: actions/checkout@v6
-      with:
-        submodules: recursive
 
     - uses: actions/cache@v4
       with:
@@ -73,19 +71,23 @@ jobs:
       run: |
         pixi run -e ${{ matrix.environment }} ccache -z
 
-    - name: Build nanoeigenpy [MacOS/Linux/Windows]
+    - name: Configure nanoeigenpy [MacOS/Linux/Windows]
       env:
         NANOEIGENPY_BUILD_TYPE: ${{ matrix.build_type }}
+      run: |
+        pixi run -e ${{ matrix.environment }} configure
+
+    - name: Build nanoeigenpy [MacOS/Linux/Windows]
       run: |
         pixi run -e ${{ matrix.environment }} build
 
     - name: Test nanoeigenpy [MacOS/Linux/Windows]
       run: |
-        pixi run -e ${{ matrix.environment }} ctest --test-dir build --output-on-failure
+        pixi run -e ${{ matrix.environment }} test
 
     - name: Install nanoeigenpy [MacOS/Linux/Windows]
       run: |
-        pixi run -e ${{ matrix.environment }} cmake --build build --target install
+        pixi run -e ${{ matrix.environment }} install
 
     - name: Show ccache statistics [MacOS/Linux/Windows]
       run: |
@@ -102,14 +104,12 @@ jobs:
 
     steps:
     - uses: actions/checkout@v6
-      with:
-        submodules: recursive
 
     - uses: prefix-dev/[email protected]
       env:
         CMAKE_BUILD_PARALLEL_LEVEL: 2
       with:
-        cache: true
+        cache: false # ⚠️ Disabling cache for testing ⚠️
         environments: test-pixi-build
 
     - name: Test package [MacOS/Linux/Windows]

.github/workflows/ros_ci.yml

@@ -31,11 +31,12 @@ jobs:
     env:
       # PRERELEASE: true  # Fails due to issues in the underlying Docker image
       BUILDER: colcon
+      VERBOSE_OUTPUT: true
+      VERBOSE_TESTS: true
+      DEBUG_BASH: true
     runs-on: ubuntu-latest
     steps:
       - uses: actions/checkout@v6
-        with:
-          submodules: recursive
       # Run industrial_ci
       - uses: 'ros-industrial/industrial_ci@ba2a3d0f830f8051b356711a8df2fedfc5d256cf'
         env: ${{ matrix.env }}

.gitignore

@@ -1,8 +1,16 @@
 build*/
+install*/
+.pytest_cache/
 .cache/
-__pycache__
-.pytest_cache
-
-# pixi environments
-.pixi
+.pixi/
+__pycache__/
+Xcode*
+*.pyc
+*~
 *.egg-info
+.ruff_cache
+.DS_Store
+compile_commands.json
+cmake-profiling.json
+result
+*.conda

.gitmodules

@@ -14,6 +14,7 @@ The format is based on [Keep a Changelog](https://keepachangelog.com/en/1.0.0/).
 - Python version update ([#25](https://github.com/Simple-Robotics/nanoeigenpy/pull/25)):
   - Project is now tested with Python 3.10 and 3.14
   - Python 3.10 is the minimal supported Python version
+- Switch to [JRL CMake modules v2](https://github.com/jrl-umi3218/jrl-cmakemodules/pull/798) ([#28](https://github.com/Simple-Robotics/nanoeigenpy/pull/28))
 
 ### Added
 - Add pixi-build support ([#25](https://github.com/Simple-Robotics/nanoeigenpy/pull/25))