From: Junio C Hamano <gitster@pobox.com>
To: Matthieu Moy <Matthieu.Moy@grenoble-inp.fr>
Cc: git@vger.kernel.org
Subject: Re: [PATCH] push: start warning upcoming default change for push.default
Date: Thu, 21 Jun 2012 10:00:51 -0700 [thread overview]
Message-ID: <7vobocsipo.fsf@alter.siamese.dyndns.org> (raw)
In-Reply-To: <vpqd34su0io.fsf@bauges.imag.fr> (Matthieu Moy's message of "Thu, 21 Jun 2012 17:50:55 +0200")
Matthieu Moy <Matthieu.Moy@grenoble-inp.fr> writes:
> Junio C Hamano <gitster@pobox.com> writes:
>
>> Perhaps we should say more to help people decide which one to choose
>> in this message.
> [...]
>> You can squelch this message by picking your preferred default now,
>> e.g. running one of these:
>>
>> git config push.default matching
>> git config push.default simple
>> git config push.default current
>
> I don't see any added value. Seeing this, it's less clear to the user,
> and he may chose 'simple', and then complain that it breaks his older
> Git.
Re-read the part you omitted with [...]. Doesn't it say something
about "only available"?
> Really, if 'simple' isn't an option for someone, why should we make
> this someone think before chosing between 'current' an 'simple'?
This is not about choosing between simple and current when both are
available. As [...] part tries to clearly state (and if it is
unclear, please clarify it further), the new default "simple" is
only available in certain versions, and 'current' is the closest
approximation for other versions. So even though there are three
choices, it is really about choosing between 'matching' (push all
relevant and current branch does not matter) and 'simple/current'
(push current branch only). We could make it like this:
e.g. running one of these:
git config push.default matching
git config push.default simple (or current)
and it may make it clearer what choices are suggested. The only
reason I made it three lines is "(or current)" would make it harder
for people who want "current only" but runs older version of git in
the same repository to cut and paste.
next prev parent reply other threads:[~2012-06-21 17:00 UTC|newest]
Thread overview: 15+ messages / expand[flat|nested] mbox.gz Atom feed top
2012-06-19 23:46 What's cooking in git.git (Jun 2012, #05; Tue, 19) Junio C Hamano
2012-06-20 12:35 ` [PATCH] push: start warning upcoming default change for push.default Matthieu Moy
2012-06-20 17:55 ` Junio C Hamano
2012-06-20 18:24 ` Matthieu Moy
2012-06-20 19:31 ` Junio C Hamano
2012-06-20 19:45 ` Matthieu Moy
2012-06-20 19:51 ` Junio C Hamano
2012-06-21 15:50 ` Matthieu Moy
2012-06-21 17:00 ` Junio C Hamano [this message]
2012-06-21 17:08 ` Matthieu Moy
2012-06-21 17:21 ` Junio C Hamano
2012-06-21 17:46 ` Junio C Hamano
2012-06-22 7:57 ` Matthieu Moy
2012-06-22 17:48 ` Junio C Hamano
2012-06-24 11:01 ` Matthieu Moy
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=7vobocsipo.fsf@alter.siamese.dyndns.org \
--to=gitster@pobox.com \
--cc=Matthieu.Moy@grenoble-inp.fr \
--cc=git@vger.kernel.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).