Skip to content

env: continue to mark non-roots as implicitly installed on partial env installs#47183

Merged
tgamblin merged 1 commit intodevelopfrom
hs/fix/environment-explicitness-regression
Oct 23, 2024
Merged

env: continue to mark non-roots as implicitly installed on partial env installs#47183
tgamblin merged 1 commit intodevelopfrom
hs/fix/environment-explicitness-regression

Conversation

@haampie
Copy link
Copy Markdown
Member

@haampie haampie commented Oct 23, 2024

Fixes a change in behavior/bug in 7041261, where partial environment
installs would mark the selected spec as explicitly installed, even if
it was not a root of the environment.

The desired behavior is that roots by definition are the to be
explicitly installed specs. The specs on the spack -e ... install x
command line are just filters for partial environment installs,
that should not influence explicit/implicit installs.

…v installs

Fixes a change in behavior/bug in
7041261, where partial environment
installs would mark the selected spec as explicitly installed, even if
it was not a root of the environment.

The desired behavior is that roots by definition are the to be
explicitly installed specs. The specs on the `spack -e ... install x`
command line are just filters for partial installs, so leave them
implicitly installed if they aren't roots.
@spackbot-app spackbot-app bot added core PR affects Spack core functionality environments tests General test capability(ies) labels Oct 23, 2024
@tgamblin tgamblin enabled auto-merge (squash) October 23, 2024 19:17
@tgamblin tgamblin merged commit 79ad6f6 into develop Oct 23, 2024
@tgamblin tgamblin deleted the hs/fix/environment-explicitness-regression branch October 23, 2024 21:17
eugeneswalker pushed a commit to eugeneswalker/spack that referenced this pull request Nov 1, 2024
…v installs (spack#47183)

Fixes a change in behavior/bug in
7041261, where partial environment
installs would mark the selected spec as explicitly installed, even if
it was not a root of the environment.

The desired behavior is that roots by definition are the to be
explicitly installed specs. The specs on the `spack -e ... install x`
command line are just filters for partial installs, so leave them
implicitly installed if they aren't roots.
eugeneswalker pushed a commit that referenced this pull request Nov 5, 2024
…v installs (#47183)

Fixes a change in behavior/bug in
7041261, where partial environment
installs would mark the selected spec as explicitly installed, even if
it was not a root of the environment.

The desired behavior is that roots by definition are the to be
explicitly installed specs. The specs on the `spack -e ... install x`
command line are just filters for partial installs, so leave them
implicitly installed if they aren't roots.
haampie added a commit that referenced this pull request Feb 18, 2025
…v installs (#47183)

Fixes a change in behavior/bug in
7041261, where partial environment
installs would mark the selected spec as explicitly installed, even if
it was not a root of the environment.

The desired behavior is that roots by definition are the to be
explicitly installed specs. The specs on the `spack -e ... install x`
command line are just filters for partial installs, so leave them
implicitly installed if they aren't roots.
@haampie haampie mentioned this pull request Feb 18, 2025
4 tasks
haampie added a commit that referenced this pull request Feb 19, 2025
…v installs (#47183)

Fixes a change in behavior/bug in
7041261, where partial environment
installs would mark the selected spec as explicitly installed, even if
it was not a root of the environment.

The desired behavior is that roots by definition are the to be
explicitly installed specs. The specs on the `spack -e ... install x`
command line are just filters for partial installs, so leave them
implicitly installed if they aren't roots.
vjranagit pushed a commit to vjranagit/spack that referenced this pull request Jan 18, 2026
…v installs (spack#47183)

Fixes a change in behavior/bug in
244b416, where partial environment
installs would mark the selected spec as explicitly installed, even if
it was not a root of the environment.

The desired behavior is that roots by definition are the to be
explicitly installed specs. The specs on the `spack -e ... install x`
command line are just filters for partial installs, so leave them
implicitly installed if they aren't roots.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

core PR affects Spack core functionality environments tests General test capability(ies)

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants