Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix Spack CI order #2370

Merged
merged 4 commits into from
Sep 21, 2022
Merged

Fix Spack CI order #2370

merged 4 commits into from
Sep 21, 2022

Conversation

garth-wells
Copy link
Member

When installing a Spack package without specifying the version,

  1. If any package is already installed, then no new version is installed
  2. If a version is specified, then the specified version is installed unless the specified version has already been installed

The CI was not working as expected because it first installed the @main version (to test development version), and then installed without a specified version (with the intention of testing the default, latest release version). However, since after install the @main version, Spack detected and existing build and did not install the release version.

This change reverse the order of install in the CI, first installing the 'default' release version and then installing @main.

@jhale jhale merged commit df50a03 into main Sep 21, 2022
@jhale jhale deleted the spack-ci-order branch September 21, 2022 07:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants