Skip to content

Seacas: Update for latest seacas releaes version#40698

Merged
sethrj merged 6 commits intospack:developfrom
gsjaardema:patch-10
Oct 31, 2023
Merged

Seacas: Update for latest seacas releaes version#40698
sethrj merged 6 commits intospack:developfrom
gsjaardema:patch-10

Conversation

@gsjaardema
Copy link
Copy Markdown
Contributor

Add option to pull latest SEACAS release; update dependency version information.

Add option to pull latest SEACAS release; update dependency version information.
Sorry for churn; need to get my eyes checked since I can't seem to see misaligned braces...
@gsjaardema gsjaardema requested review from alalazo and sethrj October 30, 2023 23:47
Comment on lines +138 to +139
depends_on("cmake@3.17:", type="build")
depends_on("cmake@3.22:", when="@2023-10-24", type="build")
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Suggested change
depends_on("cmake@3.17:", type="build")
depends_on("cmake@3.22:", when="@2023-10-24", type="build")
depends_on("cmake@3.22:", when="@2023-10-24:", type="build")
depends_on("cmake@3.17:", when="@:2023-05-30", type="build")

These should probably be ranges, and dependencies are usually ordered so newer versions are before older ones.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

done i think. Is master considered after 2023-10-24?

Comment on lines +148 to +150
depends_on("fmt@9.1.0", when="@2022-10-14")
depends_on("fmt@9.1.0:", when="@2023-05-30")
depends_on("fmt@10.1.0:", when="@2023-10-24")
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Same comments here,

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Done i think.

@gsjaardema gsjaardema requested a review from sethrj October 31, 2023 14:40
Copy link
Copy Markdown
Contributor

@sethrj sethrj left a comment

Choose a reason for hiding this comment

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

Lgtm

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.

2 participants