Skip to content

Commit d3934a1

Browse files
committed
[WIP] Use env context to set directories for post-install tests.
1 parent 669f8d4 commit d3934a1

File tree

1 file changed

+10
-5
lines changed

1 file changed

+10
-5
lines changed

.github/workflows/root-ci.yml

Lines changed: 10 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -420,6 +420,11 @@ jobs:
420420
CONTAINER_IMAGE: "registry.cern.ch/root-ci/${{ matrix.image }}:buildready" #KEEP IN SYNC WITH ABOVE
421421
CONTAINER_OPTIONS: "--security-opt label=disable --rm ${{ matrix.property == 'gpu' && '--device nvidia.com/gpu=all' || '' }}" #KEEP IN SYNC WITH ABOVE
422422

423+
env:
424+
BUILD_DIR: /github/home/ROOT-CI/build
425+
INSTALL_DIR: /github/home/ROOT-CI/install
426+
POST_INSTALL_DIR: /github/home/ROOT-CI/PostInstall
427+
423428
steps:
424429
- name: Configure large ccache
425430
if: ${{ matrix.is_special }}
@@ -564,19 +569,19 @@ jobs:
564569
ccache -s || true
565570
566571
- name: Install
567-
run: "cmake --install /github/home/ROOT-CI/build/ --prefix /github/home/ROOT-CI/install"
572+
run: "cmake --install ${{ env.BUILD_DIR }} --prefix ${{ env.INSTALL_DIR }}"
568573

569574
- name: Post-install build
570575
run: |
571-
cmake -S test/PostInstall/ -B /github/home/ROOT-CI/PostInstall -DCMAKE_PREFIX_PATH=/github/home/ROOT-CI/install;
572-
cmake --build /github/home/ROOT-CI/PostInstall;
576+
cmake -S test/PostInstall/ -B ${{ env.POST_INSTALL_DIR }} -DCMAKE_PREFIX_PATH=${{ env.INSTALL_DIR }};
577+
cmake --build ${{ env.POST_INSTALL_DIR }};
573578
574579
- name: Post-install test
575-
working-directory: /github/home/ROOT-CI/PostInstall
580+
working-directory: ${{ env.POST_INSTALL_DIR }}
576581
run: ctest -j $(nproc)
577582

578583
- name: Check installed headers
579-
run: bash test/PostInstall/check-headers.sh /github/home/ROOT-CI/install/include
584+
run: bash test/PostInstall/check-headers.sh ${{ env.INSTALL_DIR }}/include
580585

581586
event_file:
582587
# For any event that is not a PR, the CI will always run. In PRs, the CI

0 commit comments

Comments
 (0)