[prev in list] [next in list] [prev in thread] [next in thread] 

List:       git
Subject:    Re: [PATCH v6 5/7] submodule: fix segmentation fault in submodule--helper clone
From:       Jacob Keller <jacob.keller () gmail ! com>
Date:       2016-02-29 23:36:40
Message-ID: CA+P7+xpOBa5sKQyZRVpGU9kxn9281vpnF4rUYyNOeBtWiqjjSg () mail ! gmail ! com
[Download RAW message or body]

On Mon, Feb 29, 2016 at 3:20 PM, Stefan Beller <sbeller@google.com> wrote:
>
> I think this bug was put in, by "literally" translating from shell,
> see ee8838d15776, where the shell code was rewritten to C,
> specially:
>
>  git clone $quiet ${depth:+"$depth"} -n ${reference:+"$reference"} \
>      --separate-git-dir "$gitdir" "$url" "$sm_path"
>
> Anything except url and path are done optionally, but url, and path not so.
> The C code was inspired by this and aligned.
>
> This patch makes sense to me,
> Thanks,
> Stefan
>

The only thing that bothers me is that git submodule--helper clone
uses --path and --url instead of just taking two arguments after all
the optional parameters. Traditionally options are supposed to be just
that, optional. However given that long term we do not intend to keep
submodule--helper, I am not too worried about how it works here.

Thanks,
Jake
--
To unsubscribe from this list: send the line "unsubscribe git" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
[prev in list] [next in list] [prev in thread] [next in thread]