Re: [PATCH v5 5/5] worktree: copy sparse-checkout patterns and config on add

On 2/15/2022 5:01 PM, Eric Sunshine wrote:

On Mon, Feb 7, 2022 at 9:30 AM Derrick Stolee [off-list ref] wrote:

quoted

On 2/6/2022 6:30 AM, Eric Sunshine wrote:

quoted

On Mon, Jan 31, 2022 at 10:01 AM Derrick Stolee via GitGitGadget
[off-list ref] wrote:

quoted

+               git config --worktree bogus.key value &&
+               git config --unset core.bare &&

Why is this being unset? (Genuine question. Am I missing something obvious?)

I'm moving it out of the common config file. Earlier commands
enabled it in the config.worktree file for this working tree.

But won't the `git worktree add` commands which immediately follow
this bit automatically drop `core.bare=true` from the common config
file? Or am I misthinking on this? Or are you just trying to be
explicit here with the manual removal?

Ah. Here we are testing that bogus.key gets copied from the
config.worktree file, but core.bare and core.worktree do _not_.

This is kind of like the case where we run two 'git worktree add'
commands in a row. The first one moves core.bare and core.worktree
into the config.worktree file. The second one attempts to copy the
config.worktree file into the new worktree (but must filter out
these config keys).

Thanks,
-Stolee