git.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
From: Jeff King <peff@peff.net>
To: "Shawn O. Pearce" <spearce@spearce.org>
Cc: Junio C Hamano <gitster@pobox.com>,
	Paolo Bonzini <bonzini@gnu.org>,
	git@vger.kernel.org
Subject: Re: [PATCH resend] make "git push" update origin and mirrors, "git push --mirror" update mirrors
Date: Sun, 27 Apr 2008 21:21:13 -0400	[thread overview]
Message-ID: <20080428012113.GA15121@sigill.intra.peff.net> (raw)
In-Reply-To: <20080427173446.GM29771@spearce.org>

On Sun, Apr 27, 2008 at 01:34:46PM -0400, Shawn O. Pearce wrote:

> Just take the above as the rantings of someone who knows git a
> little too well, and has tried to teach it to people who don't,
> and they all have asked about the funny (to them) need for origin
> in git-pull/git-push command line sometimes (no refspecs) and not
> others (with refspecs).

I know git pretty well, and I find that particular distinction (that is,
needing to specify the remote if using refspecs, but not otherwise)
annoying.  And it _is_ a bit funny, but it has nothing to do with
concepts. It is purely a syntactic issue that relying on order of
arguments means you can't default earlier ones but specify later ones.

Whether you hit this particular syntactic funniness depends totally on
your workflow. If you don't tend to default that particular argument,
then you won't see it. But there are plenty of workflows where you
never need to specify a remote, and then typing "git push master:foo"
makes you stop and blink for a second when it fails.

Of course, I don't think there is a reasonable fix now, short of "git
push --ref master:foo".

-Peff

  parent reply	other threads:[~2008-04-28  1:22 UTC|newest]

Thread overview: 74+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2008-04-26  7:31 [PATCH resend] make "git push" update origin and mirrors, "git push --mirror" update mirrors Paolo Bonzini
2008-04-26 17:01 ` Shawn O. Pearce
2008-04-26 17:46   ` Junio C Hamano
2008-04-27  4:30     ` Shawn O. Pearce
2008-04-27  4:40       ` Shawn O. Pearce
2008-04-27  5:23         ` Junio C Hamano
2008-04-27 17:34           ` Shawn O. Pearce
2008-04-27 20:13             ` Junio C Hamano
2008-04-27 20:22               ` Paolo Bonzini
2008-04-28  1:26                 ` Jeff King
2008-04-28  5:07                   ` Paolo Bonzini
2008-04-28  9:09                     ` Jeff King
2008-04-28  9:11                       ` Jeff King
2008-04-28  9:19                         ` Paolo Bonzini
2008-04-28 10:33                           ` Johannes Schindelin
2008-04-28 11:24                             ` Paolo Bonzini
2008-04-28 11:57                               ` Johannes Schindelin
2008-04-28  3:32               ` Shawn O. Pearce
2008-04-28  5:03                 ` Paolo Bonzini
2008-04-28  6:08                 ` Stephen R. van den Berg
2008-04-28  1:21             ` Jeff King [this message]
2008-04-27  9:03         ` Paolo Bonzini
2008-04-27  7:23   ` Paolo Bonzini
2008-04-28 15:32 ` [PATCH 0/7] limit the usage of the default remote "origin" to the minimum Paolo Bonzini
2008-04-28 15:32   ` [PATCH 1/7] add special "matching refs" refspec Paolo Bonzini
2008-04-28 15:32     ` [PATCH 2/7] add push line in git-clone Paolo Bonzini
2008-04-28 15:32       ` [PATCH 3/7] Add a --push option to "git-remote add" Paolo Bonzini
2008-04-28 15:32         ` [PATCH 4/7] make "git push" update all push repositories Paolo Bonzini
2008-04-28 15:32           ` [PATCH 5/7] don't rely on zero-argument "git fetch" from within git pull Paolo Bonzini
2008-04-28 15:32             ` [PATCH 6/7] warn on "git pull" without a given branch.<name>.remote value Paolo Bonzini
2008-04-28 15:32               ` [PATCH 7/7] make "git fetch" update all fetch repositories Paolo Bonzini
2008-04-28 18:10                 ` Alex Riesen
2008-04-28 18:19                   ` Paolo Bonzini
2008-04-28 21:33                     ` Alex Riesen
2008-04-29  4:52                       ` Paolo Bonzini
2008-04-29  5:38                         ` Alex Riesen
2008-04-29  6:05                           ` Andreas Ericsson
2008-04-29  6:55                             ` Paolo Bonzini
2008-04-29 16:13                               ` Johannes Schindelin
2008-04-29 16:40                                 ` Paolo Bonzini
2008-04-29 20:34                                   ` Alex Riesen
2008-04-29  6:50                           ` Paolo Bonzini
2008-04-29  7:16                             ` Andreas Ericsson
2008-04-29  7:57                               ` Paolo Bonzini
2008-04-29  8:48                                 ` Andreas Ericsson
2008-04-29  9:02                                   ` Paolo Bonzini
2008-04-29 21:08                                   ` しらいしななこ
     [not found]                                   ` <200804292108.m3TL8moV011790@mi1.bluebottle.com>
2008-04-29 21:21                                     ` Paolo Bonzini
2008-04-29 22:21                                       ` Johannes Schindelin
2008-04-29 20:44                                 ` Alex Riesen
2008-04-29 21:15                                   ` Paolo Bonzini
2008-04-29 21:33                                     ` Alex Riesen
2008-04-29 21:41                                       ` Paolo Bonzini
2008-04-29 21:53                                         ` Alex Riesen
2008-04-29 22:26                                         ` Johannes Schindelin
2008-04-29 23:02                                           ` Jeff King
2008-04-29 23:17                                             ` Junio C Hamano
2008-04-30  5:28                                               ` Paolo Bonzini
2008-04-29 21:39                                   ` Johannes Schindelin
2008-04-29 20:24                             ` Alex Riesen
2008-05-01  6:28       ` [PATCH 2/7] add push line in git-clone Junio C Hamano
2008-05-06  8:37         ` Paolo Bonzini
2008-05-14 15:20           ` Paolo Bonzini
2008-05-14 18:16             ` Junio C Hamano
2008-05-14 19:07               ` Paolo Bonzini
2008-05-14 19:23                 ` Junio C Hamano
2008-05-14 19:40                   ` Paolo Bonzini
2008-04-30  9:23     ` [PATCH 1/7] add special "matching refs" refspec Junio C Hamano
2008-04-30  9:35       ` Paolo Bonzini
2008-04-29 19:35   ` [PATCH 0/7] limit the usage of the default remote "origin" to the minimum Jeff King
2008-04-29 21:42     ` Alex Riesen
2008-04-29 21:56     ` Junio C Hamano
2008-04-29 23:12       ` Jeff King
2008-04-30  5:24         ` Paolo Bonzini

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=20080428012113.GA15121@sigill.intra.peff.net \
    --to=peff@peff.net \
    --cc=bonzini@gnu.org \
    --cc=git@vger.kernel.org \
    --cc=gitster@pobox.com \
    --cc=spearce@spearce.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).