Git development
 help / color / mirror / Atom feed
* Re: [RFC] worktree: add --recurse-submodules support to git worktree add
@ 2026-04-15 15:12 Jimmy Aguilar Mena
  0 siblings, 0 replies; 4+ messages in thread
From: Jimmy Aguilar Mena @ 2026-04-15 15:12 UTC (permalink / raw)
  To: phillip.wood123; +Cc: git, phillip.wood, kratsbinovish


Phillip,

Good catch — let me clarify what the script actually places on disk.

The hardlink copy does not go into $GIT_COMMON_DIR/modules/ (which
both worktrees would then share). It goes into each worktree's own
gitdir entry:

cp -al $GIT_COMMON_DIR/modules \
        $GIT_COMMON_DIR/worktrees/$id/modules

So worktree A uses $GIT_COMMON_DIR/modules/sub/ and worktree B uses
$GIT_COMMON_DIR/worktrees/B/modules/sub/ — these are physically
separate directory trees with separate paths.

Within those trees the files start out as hardlinks (same inode), but
that only affects disk space accounting. It does not mean the data is
shared. Git writes files atomically by writing to a lock file and then
calling rename(2), which gives the path a new inode. So the first time
git touches HEAD, index, or any ref in one worktree's submodule
gitdir, the rename breaks that hardlink and the file becomes
independent. The other worktree's copy is untouched.

The only files that stay hardlinked forever are pack files and loose
object files — precisely because git objects are immutable by
design. Two pack files with the same content should share the same
disk blocks; there is no correctness issue.

The net effect is the same separation git worktree already provides
for the outer repo (HEAD and index under
$GIT_COMMON_DIR/worktrees/$id/, objects shared under
$GIT_COMMON_DIR/objects/), but achieved through the filesystem rather
than explicit gitdir configuration. A native --recurse-submodules
implementation should do the same thing explicitly — probably by
running the equivalent of git -C <sub-gitdir> worktree add for each
submodule, which would give each submodule its own proper worktrees/
structure rather than relying on the hardlink + rename behaviour.


Signed-off-by: Jimmy Aguilar Mena kratsbinovish@gmail.com

^ permalink raw reply	[flat|nested] 4+ messages in thread
* [RFC] worktree: add --recurse-submodules support to git worktree add
@ 2026-04-15  0:14 JAM
  2026-04-15 14:05 ` Phillip Wood
  0 siblings, 1 reply; 4+ messages in thread
From: JAM @ 2026-04-15  0:14 UTC (permalink / raw)
  To: git


[-- Attachment #1.1: Type: text/plain, Size: 2526 bytes --]

Back in 2022, Glen Choo noted [1] that git worktree add leaves submodules
unhandled and suggested a --recurse-submodules flag to fix that. The thread
went quiet. The 2015–2016 discussions by Duy and Beller [2][3] had flagged a
deeper design concern: submodule existence (which worktrees have which
submodules checked out) is tangled up with submodule configuration (URL,
path),
making per-worktree submodule support tricky to reason about.

This proposal sidesteps that concern entirely. Rather than touching
submodule
configuration at all, the idea is to reuse the git object data that's
already
present in the main repository — the same thing git worktree add already
does
for the top-level object store, extended down to the submodule layer.

The use case has also grown more pressing: multi-agent development workflows
(where several autonomous coding agents work concurrently on different
branches
of the same repository) rely heavily on worktrees for isolation, and fall
apart
on projects with submodules.

Concretely, git worktree add --recurse-submodules would:

1. Hardlink $GIT_COMMON_DIR/modules/ into the new worktree's entry.
Independent directory trees, shared inodes — no extra disk, no network.
2. Rewrite core.worktree in the hardlinked config and config.worktree
files to point at the new worktree's working directory instead of the main
repo's.
3. Run git submodule update inside the new worktree to write the .git
pointer files into each submodule directory. Entirely local since the
modules directory is already there.
4. Populate working trees with git read-tree HEAD && git checkout -- . per
submodule, since the hardlinked index files start empty.

A shell script implementing this as a prototype is attached.

The worktreeConfig extension case (step 2) is the one place that needs care,
since core.worktree may live in either config or config.worktree
depending on the submodule. The prototype handles both. The other open
question
is policy for submodules not yet initialized in the main repo — skip
silently,
warn, or error out.

Would there be interest in a proper patch series for this?

[1]
https://lore.kernel.org/git/kl6lwnimyxbq.fsf@chooglen-macbookpro.roam.corp.google.com/
[2]
https://lore.kernel.org/git/CACsJy8D8Ur4W348t-WFUPrb7SQxmff5MJ4aRp+w+ZiQ7VVvipg@mail.gmail.com/
[3]
https://lore.kernel.org/git/CAGZ79kZB8U+ERNeYpZ-i7Ldip7xbz0ND53g4bzMkzFC3pnyv+w@mail.gmail.com/

Signed-off-by: Jimmy Aguilar kratsbinovish@gmail.com

[-- Attachment #1.2: Type: text/html, Size: 3014 bytes --]

[-- Attachment #2: create-worktree.sh --]
[-- Type: application/x-shellscript, Size: 4936 bytes --]

^ permalink raw reply	[flat|nested] 4+ messages in thread

end of thread, other threads:[~2026-04-15 16:23 UTC | newest]

Thread overview: 4+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2026-04-15 15:12 [RFC] worktree: add --recurse-submodules support to git worktree add Jimmy Aguilar Mena
  -- strict thread matches above, loose matches on Subject: below --
2026-04-15  0:14 JAM
2026-04-15 14:05 ` Phillip Wood
2026-04-15 16:23   ` Junio C Hamano

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox