Skip to content

Revert "Assign priorities to configuration scopes"#49185

Merged
tgamblin merged 1 commit intodevelopfrom
revert-48420-bugfix/config-priority-based-order
Feb 25, 2025
Merged

Revert "Assign priorities to configuration scopes"#49185
tgamblin merged 1 commit intodevelopfrom
revert-48420-bugfix/config-priority-based-order

Conversation

@alalazo
Copy link
Copy Markdown
Member

@alalazo alalazo commented Feb 25, 2025

Reverts #48420

All the build jobs in pipelines are apparently relying on the bug that was fixed. The issue was not caught in the PR because generation jobs were fine, and there was nothing to rebuild.

Reverting to fix pipelines in a new PR.

@spackbot-app spackbot-app bot added core PR affects Spack core functionality documentation Improvements or additions to documentation environments gitlab Issues related to gitlab integration stand-alone-tests Stand-alone (or smoke) tests for installed packages tests General test capability(ies) utilities labels Feb 25, 2025
@alalazo alalazo self-assigned this Feb 25, 2025
@tgamblin tgamblin merged commit 9e508b0 into develop Feb 25, 2025
32 of 33 checks passed
@tgamblin tgamblin deleted the revert-48420-bugfix/config-priority-based-order branch February 25, 2025 10:33
alalazo added a commit that referenced this pull request Feb 25, 2025
white238 pushed a commit that referenced this pull request Mar 3, 2025
All the build jobs in pipelines are apparently relying on the bug that was fixed.

The issue was not caught in the PR because generation jobs were fine, and
there was nothing to rebuild.

Reverting to fix pipelines in a new PR.

This reverts commit 3ad99d7.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

core PR affects Spack core functionality documentation Improvements or additions to documentation environments gitlab Issues related to gitlab integration stand-alone-tests Stand-alone (or smoke) tests for installed packages tests General test capability(ies) utilities

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants