Skip to content

Commit d35285e

Browse files
committed
Fix: Test v5.1.0 release instead of master branch
- Reverted to default version (v5.1.0) which is what users will install - Removed incorrect assumption about parallel build issues - Caching still in place to speed up CI (~70min -> ~10min on cache hit)
1 parent 72790d4 commit d35285e

File tree

1 file changed

+3
-4
lines changed

1 file changed

+3
-4
lines changed

.github/workflows/spack.yml

Lines changed: 3 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -171,15 +171,14 @@ jobs:
171171
run: |
172172
. spack/share/spack/setup-env.sh
173173
# Install just the dependencies first (these are cached)
174-
spack install --show-log-on-error --only dependencies mfc@master~mpi~post_process
174+
spack install --show-log-on-error --only dependencies mfc~mpi~post_process
175175
timeout-minutes: 35
176176

177177
- name: Install MFC via Spack
178178
run: |
179179
. spack/share/spack/setup-env.sh
180-
# Install MFC from master branch (v5.1.0 has parallel build issues)
181-
# This step is fast since dependencies are already installed
182-
spack install --show-log-on-error mfc@master~mpi~post_process
180+
# Install MFC itself (dependencies are already installed/cached)
181+
spack install --show-log-on-error mfc~mpi~post_process
183182
timeout-minutes: 10
184183

185184
- name: Test MFC Execution

0 commit comments

Comments
 (0)