Skip to content

Commit 754ff92

Browse files
committed
[WIP] Use env context to set directories for post-install tests.
1 parent cd6dadb commit 754ff92

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

431+
env:
432+
BUILD_DIR: /github/home/ROOT-CI/build
433+
INSTALL_DIR: /github/home/ROOT-CI/install
434+
POST_INSTALL_DIR: /github/home/ROOT-CI/PostInstall
435+
431436
steps:
432437
- name: Configure large ccache
433438
if: ${{ matrix.is_special }}
@@ -572,19 +577,19 @@ jobs:
572577
ccache -s || true
573578
574579
- name: Install
575-
run: "cmake --install /github/home/ROOT-CI/build/ --prefix /github/home/ROOT-CI/install"
580+
run: "cmake --install ${{ env.BUILD_DIR }} --prefix ${{ env.INSTALL_DIR }}"
576581

577582
- name: Post-install build
578583
run: |
579-
cmake -S test/PostInstall/ -B /github/home/ROOT-CI/PostInstall -DCMAKE_PREFIX_PATH=/github/home/ROOT-CI/install;
580-
cmake --build /github/home/ROOT-CI/PostInstall;
584+
cmake -S test/PostInstall/ -B ${{ env.POST_INSTALL_DIR }} -DCMAKE_PREFIX_PATH=${{ env.INSTALL_DIR }};
585+
cmake --build ${{ env.POST_INSTALL_DIR }};
581586
582587
- name: Post-install test
583-
working-directory: /github/home/ROOT-CI/PostInstall
588+
working-directory: ${{ env.POST_INSTALL_DIR }}
584589
run: ctest -j $(nproc)
585590

586591
- name: Check installed headers
587-
run: bash test/PostInstall/check-headers.sh /github/home/ROOT-CI/install/include
592+
run: bash test/PostInstall/check-headers.sh ${{ env.INSTALL_DIR }}/include
588593

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

0 commit comments

Comments
 (0)