All of lore.kernel.org
 help / color / mirror / Atom feed
From: Junio C Hamano <gitster@pobox.com>
To: Caleb Cushing <xenoterracide@gmail.com>
Cc: Michael J Gruber <git@drmicha.warpmail.net>,
	git@vger.kernel.org, Finn Arne Gangstad <finnag@pvv.org>
Subject: Re: git push origin error (1.6.3 new default functionality)
Date: Wed, 13 May 2009 22:29:40 -0700	[thread overview]
Message-ID: <7vtz3oxnej.fsf@alter.siamese.dyndns.org> (raw)
In-Reply-To: 7vab5gz41o.fsf@alter.siamese.dyndns.org

Junio C Hamano <gitster@pobox.com> writes:

> Caleb Cushing <xenoterracide@gmail.com> writes:
>
>> On Wed, May 13, 2009 at 2:37 PM, Junio C Hamano <gitster@pobox.com> wrote:
>>> Thanks for saying this concisely, and saving me from repeating this.
>>
>> I just don't think one should have to explicitly set something to shut
>> warnings up. defaults are there for a reason. next thing you know it's
>> going to ask me if I'd like to continue, and then it will ask me to
>> press n for next.
>>
>> Why even have them?
>
> Why do you waste other people's time after repeatedly told this was
> discussed to death and everything is recoded in the list archive?

You know what is most frustrating for me with this whole thing?

As you might have guessed already, I am one of the oldest users of git, am
accustomed to the way "matching push" works, and I like it as a sensible
default behaviour for _my workflow_.  If I, Linus and you were the only
git users, there won't be these half-page-full of warning messages.

But there are others, and one of them was motivated enough to write a
patch series to introduce push.default that allows a setting that may be
more suitable than 'matching' in certain workflows, even though I may not
ever use that workflow in my projects myself.  This early vaccination
approach was the least evil solution proposed back then (which I think was
modelled after the already in-progress "deny git push from updating the
current branch" topic), and you were not around to know that I even toned
down the series not to make it too strongly suggest that the default will
change.

No, "you were not around" part is not what is frustrating.  What is
frustrating is that the original author who felt strongly enough against
'matching' default to write the patch is not defending the change in this
thread, and I have to spend time writing responses like this that I
otherwise could be using for something else to improve the project with.
And what is even more frustrating is that I cannot afford the time to
repeat the full discussion here (nor I have inclinations to), and if you
are the type who does not do his own homework, it would appear to you as
if I am all for changing the default and as if I am being unreasonable.

I do not mind appearing to be a bad guy to you or anybody per-se, but I
think people who got what they wanted earlier should come and defend the
reason why they got what they wanted.

I am nice enough not to threaten them by saying something like "since
nobody seems to be serious enough to defend this earlier change, let's
change our mind and get rid of that warning" ;-)

Oh, I already anticipate that I'll have the same frustration defending the
"deny git push from updating the current branch" that was settled eons
ago.  I am not looking forward to it.

  reply	other threads:[~2009-05-14  5:29 UTC|newest]

Thread overview: 14+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2009-05-12  1:26 git push origin error (1.6.3 new default functionality) Caleb Cushing
2009-05-12 11:11 ` Michael J Gruber
2009-05-13  5:26   ` Caleb Cushing
2009-05-13  8:32     ` Jeff King
2009-05-13  8:44       ` Johannes Sixt
2009-05-13  9:03         ` Jeff King
2009-05-13  9:54           ` Michael J Gruber
2009-05-14  6:31             ` Jeff King
2009-05-14  7:37               ` Michael J Gruber
2009-05-13 18:37   ` Junio C Hamano
2009-05-14  3:30     ` Caleb Cushing
2009-05-14  4:44       ` Junio C Hamano
2009-05-14  5:29         ` Junio C Hamano [this message]
2009-05-14  8:57           ` Finn Arne Gangstad

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=7vtz3oxnej.fsf@alter.siamese.dyndns.org \
    --to=gitster@pobox.com \
    --cc=finnag@pvv.org \
    --cc=git@drmicha.warpmail.net \
    --cc=git@vger.kernel.org \
    --cc=xenoterracide@gmail.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.