Re: [PATCH v2 3/3] push: Add the --no-recurse-submodules option
- From
Jens Lehmann <jens.lehmann@web.de>
- Date
- Jul 29, 2011, 20:19 UTC
- Message-ID
- <4E3315E6.8060501@web.de>
- In-Reply-To
- <7vy5zikw6e.fsf@alter.siamese.dyndns.org>
Am 28.07.2011 22:05, schrieb Junio C Hamano:
Show 16 quoted lines
> Fredrik Gustafsson <iveqy@iveqy.com> writes:
>> diff --git a/transport.h b/transport.h
>> index 161d724..c6ccf8c 100644
>> --- a/transport.h
>> +++ b/transport.h
>> @@ -101,6 +101,7 @@ struct transport {
>> #define TRANSPORT_PUSH_MIRROR 8
>> #define TRANSPORT_PUSH_PORCELAIN 16
>> #define TRANSPORT_PUSH_SET_UPSTREAM 32
>> +#define TRANSPORT_PUSH_NO_RECURSE_SUBMODULES 64
>
> Also naming the constant with NO will invite an unnecessary double
> negation, like this:
>
> if (!(flags & FROTZ_NO_NITFOL))
> do_nitfol_to_frotz();Right, we'll drop the "NO_" from that constant.
> Besides, I would be moderately annoyed if this check were the default.
We will skip the check for submodules without remotes, does that lessen your annoyance? We still think it is a good idea to have that test enabled by default, but it might be a good idea to wait with that until we provide a central config option to enable users to turn that off with a simple "git config push.recurseSubmodules off". What do you think?