Skip to content

axom: Add cxxstd variant for C++20, python variant improvements - #5791

Merged
becker33 merged 4 commits into
spack:developfrom
bmhan12:feature/han12/c_20
Aug 13, 2026
Merged

axom: Add cxxstd variant for C++20, python variant improvements#5791
becker33 merged 4 commits into
spack:developfrom
bmhan12:feature/han12/c_20

Conversation

@bmhan12

@bmhan12 bmhan12 commented Jul 29, 2026

Copy link
Copy Markdown
Contributor

This PR:

Relates to:

@bmhan12 bmhan12 changed the title axom: Add cxxstd variant for C++20 axom: Add cxxstd variant for C++20, python variant improvements Jul 29, 2026
@spackbot-triage spackbot-triage Bot added dependencies Modifications with a `depends_on()` directive update-package Modifications to packages in the repository conflicts Modifications with a `conflicts()` directive new-variant Modifications to packages' `variant()` directives stand-alone-tests Stand-alone (or smoke) tests for installed packages extends labels Jul 29, 2026
@spackbot-triage
spackbot-triage Bot requested a review from white238 July 29, 2026 16:42
@dtaller

dtaller commented Jul 29, 2026

Copy link
Copy Markdown
Contributor

Thanks! I appreciate it!

@bmhan12 bmhan12 mentioned this pull request Jul 29, 2026
Merged
dtaller
dtaller reviewed Jul 29, 2026
View reviewed changes
Comment thread repos/spack_repo/builtin/packages/axom/package.py
@bernhardkaindl bernhardkaindl mentioned this pull request Aug 6, 2026
Merged
becker33
becker33 requested changes Aug 12, 2026
View reviewed changes
Comment thread repos/spack_repo/builtin/packages/axom/package.py Outdated
Comment thread repos/spack_repo/builtin/packages/axom/package.py Outdated
@bmhan12
bmhan12 force-pushed the feature/han12/c_20 branch from c8105bb to e7f39d3 Compare August 12, 2026 22:54
@becker33
becker33 merged commit b3674ac into spack:develop Aug 13, 2026
23 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@becker33 becker33 becker33 approved these changes
@white238 white238 Awaiting requested review from white238
+1 more reviewer
@dtaller dtaller dtaller approved these changes
Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

conflicts Modifications with a `conflicts()` directive dependencies Modifications with a `depends_on()` directive extends new-variant Modifications to packages' `variant()` directives stand-alone-tests Stand-alone (or smoke) tests for installed packages update-package Modifications to packages in the repository

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@bmhan12 @dtaller @becker33