Skip to content

Conversation

@wdconinc
Copy link
Contributor

Briefly, what does this PR introduce?

This PR removes the 'concretizer: unify: when_possible' configuration from spack.yaml for xl and cuda.

What kind of change does this PR introduce?

  • Bug fix (issue: unneeded)
  • New feature (issue #__)
  • Documentation update
  • Other: __

Removed the 'concretizer' configuration from spack.yaml.
Copilot AI review requested due to automatic review settings November 17, 2025 18:53
@wdconinc wdconinc requested a review from veprbl November 17, 2025 18:54
@wdconinc wdconinc enabled auto-merge (squash) November 17, 2025 18:54
Copilot finished reviewing on behalf of wdconinc November 17, 2025 18:54
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull Request Overview

This PR removes the 'concretizer: unify: when_possible' configuration from the xl/spack.yaml file. This setting was previously used to allow multiple epic versions, but is apparently no longer needed.

  • Removes the concretizer configuration block from spack-environment/xl/spack.yaml

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

veprbl
veprbl previously approved these changes Nov 17, 2025
@veprbl veprbl dismissed their stale review November 17, 2025 19:46

Looks like spack is not happy.

@wdconinc
Copy link
Contributor Author

Looks like spack is not happy.

Seems like eic_cuda had some other duplication. I'll have to see if it's bad or can be remedied.

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.

3 participants