Stefan Beller <[email protected]> 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?
--
To unsubscribe from this list: send the line "unsubscribe git" in
the body of a message to [email protected]
More majordomo info at http://vger.kernel.org/majordomo-info.html
