Skip to content

Commit 3ea9984

Browse files
committed
[WIP] Use env context to set directories for post-install tests.
1 parent afc4bbd commit 3ea9984

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
@@ -440,6 +440,11 @@ jobs:
440440
CONTAINER_IMAGE: "registry.cern.ch/root-ci/${{ matrix.image }}:buildready" #KEEP IN SYNC WITH ABOVE
441441
CONTAINER_OPTIONS: "--security-opt label=disable --rm ${{ matrix.property == 'gpu' && '--device nvidia.com/gpu=all' || '' }}" #KEEP IN SYNC WITH ABOVE
442442

443+
env:
444+
BUILD_DIR: /github/home/ROOT-CI/build
445+
INSTALL_DIR: /github/home/ROOT-CI/install
446+
POST_INSTALL_DIR: /github/home/ROOT-CI/PostInstall
447+
443448
steps:
444449
- name: Configure large ccache
445450
if: ${{ matrix.is_special }}
@@ -585,19 +590,19 @@ jobs:
585590
ccache -s || true
586591
587592
- name: Install
588-
run: "cmake --install /github/home/ROOT-CI/build/ --prefix /github/home/ROOT-CI/install"
593+
run: "cmake --install ${{ env.BUILD_DIR }} --prefix ${{ env.INSTALL_DIR }}"
589594

590595
- name: Post-install build
591596
run: |
592-
cmake -S test/PostInstall/ -B /github/home/ROOT-CI/PostInstall -DCMAKE_PREFIX_PATH=/github/home/ROOT-CI/install;
593-
cmake --build /github/home/ROOT-CI/PostInstall;
597+
cmake -S test/PostInstall/ -B ${{ env.POST_INSTALL_DIR }} -DCMAKE_PREFIX_PATH=${{ env.INSTALL_DIR }};
598+
cmake --build ${{ env.POST_INSTALL_DIR }};
594599
595600
- name: Post-install test
596-
working-directory: /github/home/ROOT-CI/PostInstall
601+
working-directory: ${{ env.POST_INSTALL_DIR }}
597602
run: ctest -j $(nproc)
598603

599604
- name: Check installed headers
600-
run: bash test/PostInstall/check-headers.sh /github/home/ROOT-CI/install/include
605+
run: bash test/PostInstall/check-headers.sh ${{ env.INSTALL_DIR }}/include
601606

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

0 commit comments

Comments
 (0)