Skip to content

Package requirements: support cflags (attempt 2)#37584

Merged
alalazo merged 5 commits intospack:developfrom
scheibelp:cflags-pkg-requirements2
May 11, 2023
Merged

Package requirements: support cflags (attempt 2)#37584
alalazo merged 5 commits intospack:developfrom
scheibelp:cflags-pkg-requirements2

Conversation

@scheibelp
Copy link
Copy Markdown
Member

@scheibelp scheibelp commented May 9, 2023

Fixes #37163
Fixes #34241

Temporary alternative to #37180

Adds choice rules for compiler flags specifically, allowing solver to set compiler flags introduced by requirements.

@spackbot-app spackbot-app bot added core PR affects Spack core functionality tests General test capability(ies) labels May 9, 2023
@scheibelp scheibelp requested a review from alalazo May 10, 2023 18:51
@alalazo alalazo added this to the v0.20.0 milestone May 10, 2023
@alalazo alalazo self-assigned this May 10, 2023
@alalazo alalazo merged commit bfa54da into spack:develop May 11, 2023
alalazo added a commit to alalazo/spack that referenced this pull request Aug 23, 2023
alalazo added a commit to alalazo/spack that referenced this pull request Aug 24, 2023
alalazo added a commit to alalazo/spack that referenced this pull request Oct 16, 2023
alalazo added a commit to alalazo/spack that referenced this pull request Oct 17, 2023
alalazo added a commit to alalazo/spack that referenced this pull request Oct 18, 2023
alalazo added a commit to alalazo/spack that referenced this pull request Oct 18, 2023
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 tests General test capability(ies)

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Package requirements: failure with compiler flags require does not work with cflags=

2 participants