From: Eric Sunshine <sunshine@sunshineco.com>
Date: Wed, 23 Mar 2016 17:43:13 GMT
Subject: Re: [PATCH] add option -n (--no-checkout) to git-worktree add
Message-ID: <CAPig+cTRa5YmbHdbXFNKNPN3GKpGOBUdVk774u9446J1qnYpfA@mail.gmail.com>
In-Reply-To: <xmqq7fgtdwgb.fsf@gitster.mtv.corp.google.com>

On Wed, Mar 23, 2016 at 11:51 AM, Junio C Hamano <gitster@pobox.com> wrote:
> Ray Zhang <zhanglei002@gmail.com> writes:
>
>> @@ -320,6 +323,7 @@ static int add(int ac, const char **av, const char *prefix)
>>               OPT_STRING('B', NULL, &new_branch_force, N_("branch"),
>>                          N_("create or reset a branch")),
>>               OPT_BOOL(0, "detach", &opts.detach, N_("detach HEAD at named commit")),
>> +             OPT_BOOL('n', "no-checkout", &opts.no_checkout, N_("don't create a checkout")),
>
> This would allow --no-no-checkout, which is idiotic, wouldn't it?
>
> How about
>
>     OPT_BOOL(0, "checkout", &opts.checkout, N_("populate the new working tree"))
>
> and set opts.checkout to true when initializing?

I think this code was copied verbatim from builtin/clone.c, and, as a
newcomer to the project, it's understandable that Ray Zhang imitated
existing code, but I agree that it would be better to avoid repeating
the misbehavior.