Skip to content

Fix compiling dealii with trilinos@master#15288

Merged
adamjstewart merged 1 commit intospack:developfrom
mathsen:dealii_gotype
Mar 3, 2020
Merged

Fix compiling dealii with trilinos@master#15288
adamjstewart merged 1 commit intospack:developfrom
mathsen:dealii_gotype

Conversation

@mathsen
Copy link
Copy Markdown
Contributor

@mathsen mathsen commented Mar 2, 2020

cherry-picking global ordinal type variant selector from this pull request and setting dealii to GO int, to make it compile again

…ng dealii to GO int, to make it compile again
Copy link
Copy Markdown
Member

@adamjstewart adamjstewart left a comment

Choose a reason for hiding this comment

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

depends_on('slepc@:3.6.3', when='@:8.4.1+slepc+petsc+mpi')
depends_on('slepc~arpack', when='+slepc+petsc+mpi+int64')
depends_on('sundials@:3~pthread', when='@9.0:+sundials')
depends_on('trilinos gotype=int', when='+trilinos')
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

i don't think Spack can combine depends_on, you should probably add this to each depends_on below

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Spack merges all depends_on constraints, so this should be fine

@aprokop
Copy link
Copy Markdown
Contributor

aprokop commented Mar 10, 2020

How come this was merged completely ignoring the whole discussion and reasons for not merging it in #14215??? This is frustrating. It also apparently broke things: #15423.

@mathsen mathsen mentioned this pull request Mar 12, 2020
2 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants