feat: Update multistage Dockerfiles to adjust build and test order, and add optional Python bindings copying

Author: actions-user

feelpp-mor/Dockerfile.multistage

@@ -58,18 +58,18 @@ RUN git config --global user.name "Docker Robot" && \
 
 WORKDIR /src/feelpp
 
-# Configure, build, test, install, and cleanup in a SINGLE layer to minimize image size
-# Tests run BEFORE install (they use build paths, not install paths)
+# Configure, build, install, test, and cleanup in a SINGLE layer to minimize image size
+# Install runs BEFORE tests so tests use installed paths
 RUN cmake --preset mor \
         -DCMAKE_BUILD_TYPE=Release \
         -DCMAKE_INSTALL_PREFIX=/usr/local \
         -DCMAKE_CXX_COMPILER=${CXX} \
         -DCMAKE_C_COMPILER=${CC} \
         ${CMAKE_FLAGS} && \
     cmake --build --preset mor && \
-    (ctest --preset mor || true) && \
-    cmake --install build/mor && \
+    cmake --build --preset mor -t install && \
     ldconfig && \
+    (ctest --preset mor || true) && \
     rm -rf build/mor /root/feelppdb /root/.cache /feel
 
 # =============================================================================
@@ -108,11 +108,17 @@ COPY --from=builder /usr/local/include/feelpp/feel/feelmor /usr/local/include/fe
 # Copy MOR CMake and data files
 COPY --from=builder /usr/local/share/feelpp/mor /usr/local/share/feelpp/mor
 
+# Copy Python bindings for MOR (optional, may not exist if Python disabled)
+RUN --mount=from=builder,source=/usr/local,target=/mnt/builder \
+    mkdir -p /usr/local/lib/python3/dist-packages && \
+    cp -r /mnt/builder/lib/python3/dist-packages/feelpp* /usr/local/lib/python3/dist-packages/ 2>/dev/null || true
+
 # Update library cache
 RUN ldconfig
 
 USER feelpp
 ENV HOME=/home/feelpp
 ENV FEELPP_REPOSITORY=/feel
+ENV PYTHONPATH=/usr/local/lib/python3/dist-packages:${PYTHONPATH}
 WORKDIR ${HOME}
 CMD ["/bin/bash"]

feelpp-toolboxes/Dockerfile.multistage

@@ -58,18 +58,18 @@ RUN git config --global user.name "Docker Robot" && \
 
 WORKDIR /src/feelpp
 
-# Configure, build, test, install, and cleanup in a SINGLE layer to minimize image size
-# Tests run BEFORE install (they use build paths, not install paths)
+# Configure, build, install, test, and cleanup in a SINGLE layer to minimize image size
+# Install runs BEFORE tests so tests use installed paths
 RUN cmake --preset toolboxes \
         -DCMAKE_BUILD_TYPE=Release \
         -DCMAKE_INSTALL_PREFIX=/usr/local \
         -DCMAKE_CXX_COMPILER=${CXX} \
         -DCMAKE_C_COMPILER=${CC} \
         ${CMAKE_FLAGS} && \
     cmake --build --preset toolboxes && \
-    (ctest --preset toolboxes || true) && \
-    cmake --install build/toolboxes && \
+    cmake --build --preset toolboxes -t install && \
     ldconfig && \
+    (ctest --preset toolboxes || true) && \
     rm -rf build/toolboxes /root/feelppdb /root/.cache /feel
 
 # =============================================================================
@@ -100,11 +100,17 @@ RUN --mount=from=builder,source=/usr/local,target=/mnt/builder \
     cp /mnt/builder/bin/feelpp_toolbox* /usr/local/bin/ 2>/dev/null || true && \
     cp /mnt/builder/lib/libfeelpp_toolbox* /usr/local/lib/ 2>/dev/null || true
 
+# Copy Python bindings for toolboxes (optional, may not exist if Python disabled)
+RUN --mount=from=builder,source=/usr/local,target=/mnt/builder \
+    mkdir -p /usr/local/lib/python3/dist-packages && \
+    cp -r /mnt/builder/lib/python3/dist-packages/feelpp* /usr/local/lib/python3/dist-packages/ 2>/dev/null || true
+
 # Update library cache
 RUN ldconfig
 
 USER feelpp
 ENV HOME=/home/feelpp
 ENV FEELPP_REPOSITORY=/feel
+ENV PYTHONPATH=/usr/local/lib/python3/dist-packages:${PYTHONPATH}
 WORKDIR ${HOME}
 CMD ["/bin/bash"]

feelpp/Dockerfile.full

@@ -60,8 +60,8 @@ RUN git config --global user.name "Docker Robot" && \
 
 WORKDIR /src/feelpp
 
-# Configure, build, test, install, and cleanup in a SINGLE layer to minimize image size
-# Tests run BEFORE install (they use build paths, not install paths)
+# Configure, build, install, test, and cleanup in a SINGLE layer to minimize image size
+# Install runs BEFORE tests so tests use installed paths
 RUN cmake --preset default \
         -DCMAKE_INSTALL_PREFIX=/usr/local \
         -DCMAKE_BUILD_TYPE=Release \
@@ -70,9 +70,9 @@ RUN cmake --preset default \
         -DFEELPP_ENABLE_FEELPP_PYTHON=ON \
         ${CMAKE_FLAGS} && \
     cmake --build --preset default && \
-    ctest --preset default || true && \
-    cmake --install build/default && \
+    cmake --build --preset default -t install && \
     ldconfig && \
+    ctest --preset default || true && \
     rm -rf build/default /root/feelppdb /root/.cache /feel
 
 # =============================================================================

feelpp/Dockerfile.multistage

@@ -54,18 +54,18 @@ RUN git config --global user.name "Docker Robot" && \
 
 WORKDIR /src/feelpp
 
-# Configure, build, test, install, and cleanup in a SINGLE layer to minimize image size
-# Tests run BEFORE install (they use build paths, not install paths)
+# Configure, build, install, test, and cleanup in a SINGLE layer to minimize image size
+# Install runs BEFORE tests so tests use installed paths
 RUN cmake --preset feelpp \
         -DCMAKE_INSTALL_PREFIX=/usr/local \
         -DCMAKE_BUILD_TYPE=Release \
         -DCMAKE_CXX_COMPILER=${CXX} \
         -DCMAKE_C_COMPILER=${CC} \
         ${CMAKE_FLAGS} && \
     cmake --build --preset feelpp && \
-    ctest --preset feelpp || true && \
-    cmake --install build/feelpp && \
+    cmake --build --preset feelpp -t install && \
     ldconfig && \
+    ctest --preset feelpp || true && \
     rm -rf build/feelpp /root/feelppdb /root/.cache /feel
 
 # =============================================================================