Skip to content

Commit

Permalink
ci: remove any mention of ROOT for consumer tests
Browse files Browse the repository at this point in the history
It seems some jobs include ROOT in DYLD_LIBRARY_PATH, while others
don't, even if those jobs are part of the same matrix. Not sure
what this correlates to, but jobs which have ROOT in DYLD_LIBRARY_PATH
tend to succeed, whereas those that don't tend to fail. In any case,
we shouldn't need ROOT for consumer build tests, which lately have
been failing the most often.
  • Loading branch information
c-dilks committed Apr 10, 2024
1 parent e9c93e5 commit 3174dc4
Showing 1 changed file with 0 additions and 8 deletions.
8 changes: 0 additions & 8 deletions .github/workflows/ci.yml
Original file line number Diff line number Diff line change
Expand Up @@ -497,7 +497,6 @@ jobs:
needs:
- download_test_data
- build_hipo
- build_root
- build_iguana
runs-on: ${{ inputs.runner }}
container:
Expand All @@ -520,11 +519,6 @@ jobs:
with:
key: ${{ needs.build_hipo.outputs.key }}
path: hipo.tar.zst
- name: get local dependency `ROOT`
uses: actions/cache/restore@v4
with:
key: ${{ needs.build_root.outputs.key }}
path: root.tar.zst
- name: get iguana build artifacts
uses: actions/download-artifact@v4
with:
Expand All @@ -542,8 +536,6 @@ jobs:
- name: set cmake prefix path
if: ${{ matrix.tool == 'cmake' }}
run: echo CMAKE_PREFIX_PATH=$(pwd)/hipo >> $GITHUB_ENV
- name: source environment for ROOT
run: .github/source-ROOT.sh root ${{ inputs.runner }} ld # needed for `ld` path
- name: source iguana environment
run: source iguana/bin/this_iguana.sh --verbose --githubCI
### build test
Expand Down

0 comments on commit 3174dc4

Please sign in to comment.