Skip to content
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

Allow empty spack environments when constructing hash #336

Merged

Conversation

douglasjacobsen
Copy link
Collaborator

When building an experiment / workspace inventory, previously it was required that if an environment had a spack.yaml, it also created a spack.lock (or Ramble would error).

This merge allows spack.yaml files that have no packages (And thus, won't concretized into a spack.lock) without throwing an error.

@douglasjacobsen douglasjacobsen added the enhancement New feature or request label Dec 6, 2023
@douglasjacobsen douglasjacobsen requested a review from rfbgo December 6, 2023 21:56
rfbgo
rfbgo previously approved these changes Dec 6, 2023
When building an experiment / workspace inventory, previously it was
required that if an environment had a spack.yaml, it also created a
spack.lock (or Ramble would error).

This commit allows spack.yaml files that have no packages (And thus,
won't concretized into a spack.lock) without throwing an error.
@rfbgo rfbgo merged commit 9739905 into GoogleCloudPlatform:develop Dec 7, 2023
5 checks passed
@douglasjacobsen douglasjacobsen deleted the fix_empty_spack_errors branch December 8, 2023 19:13
@rfbgo rfbgo mentioned this pull request Dec 13, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants