Skip to content

fix(pkg): extract submodules#12046

Merged
rgrinberg merged 1 commit intomainfrom
ps/rr/fix_pkg___extract_submodules
Jul 20, 2025
Merged

fix(pkg): extract submodules#12046
rgrinberg merged 1 commit intomainfrom
ps/rr/fix_pkg___extract_submodules

Conversation

@rgrinberg
Copy link
Copy Markdown
Member

[$ git archive] does not respect submodules. Therefore we need to run it for every submodule ourselves.

Fixes #11606

@rgrinberg rgrinberg requested a review from Alizter July 19, 2025 16:48
@rgrinberg rgrinberg force-pushed the ps/rr/fix_pkg___extract_submodules branch from 91f1f54 to 6fb04db Compare July 19, 2025 16:52
Copy link
Copy Markdown
Collaborator

@Alizter Alizter left a comment

Choose a reason for hiding this comment

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

Seems ok. This addresses the TODO on line 759 by the way.

@rgrinberg rgrinberg force-pushed the ps/rr/fix_pkg___extract_submodules branch from 6fb04db to 1cf9d51 Compare July 20, 2025 13:18
[$ git archive] does not respect submodules. Therefore we need to run it
for every submodule ourselves.

Signed-off-by: Rudi Grinberg <me@rgrinberg.com>

<!-- ps-id: 2f35fda7-d9da-43e3-86a3-2eeb003e7a9c -->

Signed-off-by: Rudi Grinberg <me@rgrinberg.com>
@rgrinberg rgrinberg merged commit fd307df into main Jul 20, 2025
23 of 25 checks passed
@rgrinberg rgrinberg deleted the ps/rr/fix_pkg___extract_submodules branch July 20, 2025 13:18
Sudha247 pushed a commit to Sudha247/dune that referenced this pull request Jul 23, 2025
[$ git archive] does not respect submodules. Therefore we need to run it
for every submodule ourselves.

Signed-off-by: Rudi Grinberg <me@rgrinberg.com>
Leonidas-from-XIV added a commit that referenced this pull request Dec 23, 2025
While going through the tests looking at the usages of `sed` I noticed
that this test works now (fixed in #12046) so I've updated the wording
and removed `sed`.

Signed-off-by: Marek Kubica <marek@tarides.com>
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.

Dune package management: submodules

2 participants