All of lore.kernel.org
 help / color / mirror / Atom feed
From: Matthieu Moy <Matthieu.Moy@grenoble-inp.fr>
To: Junio C Hamano <gitster@pobox.com>
Cc: git@vger.kernel.org
Subject: Re: [PATCH] push: start warning upcoming default change for push.default
Date: Thu, 21 Jun 2012 17:50:55 +0200	[thread overview]
Message-ID: <vpqd34su0io.fsf@bauges.imag.fr> (raw)
In-Reply-To: <7v62alvk2e.fsf@alter.siamese.dyndns.org> (Junio C. Hamano's message of "Wed, 20 Jun 2012 12:51:05 -0700")

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. Really, if 'simple' isn't an option for someone, why should we make
this someone think before chosing between 'current' an 'simple'?

Also, my version purposely made 'simple' more visible than 'current'
(cut-and-paste ready alone on its line Vs within parenthesis), because
this is the one we want to advertise. Putting the 3 options at the same
level is a regression to me.

We can rephrase the advice to make it clearer that 'simple' !=
'current', like

  (the 'simple' mode was introduced in Git 1.7.11. Use the similar mode
  'current' instead if you sometimes use older versions of Git)

-- 
Matthieu Moy
http://www-verimag.imag.fr/~moy/

  reply	other threads:[~2012-06-21 15:54 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 [this message]
2012-06-21 17:00             ` Junio C Hamano
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=vpqd34su0io.fsf@bauges.imag.fr \
    --to=matthieu.moy@grenoble-inp.fr \
    --cc=git@vger.kernel.org \
    --cc=gitster@pobox.com \
    /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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.