-
Notifications
You must be signed in to change notification settings - Fork 15
refactor: Consolidate shared package versions for spack.yaml files #309
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Copilot encountered an error and was unable to review this pull request. You can try again by re-requesting a review.
|
@bmhan12 When we try to build the TPL's on LC we get the following error: I think this means that the two new yaml files were not copied to the install directory. I.e. they are not present in who should be responsible for copying these? I assume it is the same component that copies |
This PR does require an update to the uberenv submodule to use the versions and defaults. |
|
@bmhan12 well...that is embarrassing |
This PR:
versions.yamlanddefaults.yamlto spack environment llnl/uberenv#141versions.yaml, addressing one of the issues discussed in refactor: Complete spack migration & move versions to geos package.py. #305, where the same version information does not need to be specified in everyspack.yamlfile.See GEOS-DEV/GEOS#3697 for testing.