Skip to content

feat: spack: views: collapse unique subtrees in symlink case#210

Open
wdconinc wants to merge 2 commits intomasterfrom
spack-views-collapse-subtrees
Open

feat: spack: views: collapse unique subtrees in symlink case#210
wdconinc wants to merge 2 commits intomasterfrom
spack-views-collapse-subtrees

Conversation

@wdconinc
Copy link
Contributor

Briefly, what does this PR introduce?

This PR applies spack/spack#52135 to reduce the view size in terms of file count.

Copilot AI review requested due to automatic review settings March 24, 2026 15:13
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR updates the pinned Spack cherry-picks used by the container build pipeline to include the upstream Spack change that collapses unique subtrees for symlink views, reducing the resulting view file count.

Changes:

  • Add Spack commit d55f9abacc4e153d43a4c8bff81a383ba2f311f5 to SPACK_CHERRYPICKS.
  • Document the new cherry-pick in the inline reference list.

@wdconinc
Copy link
Contributor Author

@copilot open a new pull request to apply changes based on the comments in this thread. Indicate in docs/spack-environment.md that the list of commits for spack and spack-packages are illustrative only and may be different in reality.

Copy link
Contributor

Copilot AI commented Mar 24, 2026

@wdconinc I've opened a new pull request, #211, to work on those changes. Once the pull request is ready, I'll request review from you.

…211)

* Initial plan

* docs: clarify cherry-pick lists in spack-environment.md are illustrative

Co-authored-by: wdconinc <4656391+wdconinc@users.noreply.github.com>
Agent-Logs-Url: https://github.com/eic/containers/sessions/55eaa520-04c7-42a9-ac0b-07b0bf4ff6f0

---------

Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com>
Co-authored-by: wdconinc <4656391+wdconinc@users.noreply.github.com>
@wdconinc
Copy link
Contributor Author

This PR saves us 135k symlinks in eic_ci.

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.

4 participants