On Wed, Jan 20, 2016 at 4:08 PM, Junio C Hamano <gitster@xxxxxxxxx> wrote:
> Stefan Beller <sbeller@xxxxxxxxxx> writes:
>
>>>> @@ -165,6 +166,10 @@ cmd_add()
>>>> --depth=*)
>>>> depth=$1
>>>> ;;
>>>> + -g|--group)
>>>> + submodule_groups=${submodule_groups:+${submodule_groups};}"$2"
>>>> + shift
>>>> + ;;
>>>
>>> You would want to accept "--group=<name>" as well, just like
>>> existing --reference and --depth do. It won't be much more code,
>>> and when you move to C (hence parse_options) you'd get it for free
>>> anyway.
>>
>> I am not sure, if I will to move `add` to C any time soon. Sure I desire
>> less shell and more C[1], but I'd think my time could be spent better than
>> just converting scripts to C. Sometimes I have to though, such as in the
>> case of `init` as the the call out from C to shell is too ugly and the effort to
>> do that is not that much less.
>
> You can do so in less time than you spent making the above 5-line
> excuse. It won't be much more code, and it is not ugly at all.
>
> --group=*)
> group=$group ${1#--group=} ;;
>
> or something, right?
Right, that's what I have here now. I should have said that.
I was not trying to excuse anything, but more explaining the situation
when reading the second sentence. (You seem to assume I'd want to rewrite
all the shell scripts. Which I am not)
--
To unsubscribe from this list: send the line "unsubscribe git" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html