Skip to content

spack clean: all means all#50984

Merged
tgamblin merged 3 commits intodevelopfrom
bugfix/all-means-all
Nov 4, 2025
Merged

spack clean: all means all#50984
tgamblin merged 3 commits intodevelopfrom
bugfix/all-means-all

Conversation

@becker33
Copy link
Copy Markdown
Member

@becker33 becker33 commented Jul 9, 2025

Currently, spack clean -a doesn't do spack clean -b. This is turning into a huge footgun for users who expect "all" to mean "all".

This PR makes spack clean -a be equivalent to throwing all other feature flags for spack clean

@alalazo alalazo added this to the v1.1.0 milestone Jul 28, 2025
Signed-off-by: Gregory Becker <[email protected]>
Signed-off-by: Gregory Becker <[email protected]>
@becker33 becker33 force-pushed the bugfix/all-means-all branch from de4a0ca to a9fbf81 Compare November 3, 2025 22:18
tgamblin
tgamblin previously approved these changes Nov 3, 2025
@tgamblin tgamblin enabled auto-merge (squash) November 3, 2025 22:35
@tgamblin tgamblin merged commit 4b9d4a7 into develop Nov 4, 2025
31 of 32 checks passed
@tgamblin tgamblin deleted the bugfix/all-means-all branch November 4, 2025 00:53
hippo91 pushed a commit to hippo91/spack that referenced this pull request Nov 4, 2025
Currently, `spack clean -a` doesn't do `spack clean -b`.
This is turning into a huge footgun for users who 
expect "all" to mean "all".

This PR makes `spack clean -a` be equivalent to throwing
all other feature flags for `spack clean`

---------

Signed-off-by: Gregory Becker <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants