All of lore.kernel.org
 help / color / mirror / Atom feed
From: Finn Arne Gangstad <finnag@pvv.org>
To: Paolo Bonzini <bonzini@gnu.org>
Cc: git@vger.kernel.org
Subject: Re: push.default???
Date: Tue, 23 Jun 2009 12:34:28 +0200	[thread overview]
Message-ID: <20090623103428.GA4214@pvv.org> (raw)
In-Reply-To: <h1nks1$vdl$1@ger.gmane.org>

On Mon, Jun 22, 2009 at 12:02:33PM +0200, Paolo Bonzini wrote:
> Hi all, I just upgraded to git 1.6.3 and found this new little gem  
> called push.default...
> [...]

You should have been here when we discussed this! :)

> 1) Also in 1.6.3, invent a special refspec for "tracking", something  
> like "HEAD>" (of course this is not a special case; "refs/heads/*>"  
> would also work, yadda yadda)

Yes, this is a weakness righ now - the only way to get tracking
semantics is to set push.default. I could not find a very good way of
specifying this. We currently have the magic refspecs : and
HEAD. Adding a ">" to "HEAD>" would be annoying I think, since it has
to be quoted in the shell.

Maybe we can use ":" as an escape, it is not allowed in refspecs.
Something like "::tracking" (and we cold also have "::matching",
"::current" and so on for completeness)

> 2) Also in 1.6.3, add a "--push={current,tracking,matching,mirror}"  
> option to "git remote add" that would set up a push refspec without the  
> need to actually know refspec syntax. (--mirror would become just a  
> synonym for --push=mirror).

Sounds like a good idea, the options would also make sense to push I think,
so you can "git push [--current|tracking|...] ".

> 3) Possibly, in 1.6.3 make "git clone" add a "push = :" line for the  
> origin branch.  This was actually suggested in a patch by myself.

This would destroy the intention of my patch, it would render the
configuration variable pointless I think (and would also silently push
matching).

> 4) in 1.6.4 or 1.7.0, make "git push" fail outright if there is no push  
> line, with text suggesting [...]

Hopefully we can get to this stage, that a unconfigured "git push"
gives a small message, indicating how to configure it, and not push
anything. Most "oldtimers" should have configured this already, so it
should not break many setups.

- Finn Arne

  parent reply	other threads:[~2009-06-23 10:56 UTC|newest]

Thread overview: 18+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2009-06-22 10:02 push.default??? Paolo Bonzini
2009-06-22 16:31 ` push.default??? Junio C Hamano
2009-06-22 17:55   ` push.default??? Paolo Bonzini
2009-06-23 10:34 ` Finn Arne Gangstad [this message]
2009-06-23 12:59   ` push.default??? Paolo Bonzini
2009-06-23 13:11     ` push.default??? Finn Arne Gangstad
2009-06-23 13:21       ` push.default??? Andreas Ericsson
2009-06-23 13:57         ` push.default??? Finn Arne Gangstad
2009-06-23 14:07           ` push.default??? Andreas Ericsson
2009-06-23 13:28       ` push.default??? Paolo Bonzini
2009-06-23 14:48         ` push.default??? Finn Arne Gangstad
2009-06-23 16:32           ` push.default??? Paolo Bonzini
2009-06-23 17:51             ` push.default??? Junio C Hamano
2009-06-23 17:59               ` push.default??? Junio C Hamano
2009-06-24  5:50               ` push.default??? Miles Bader
2009-06-24  6:35                 ` push.default??? Junio C Hamano
2009-06-24  8:50               ` push.default??? Paolo Bonzini
2009-06-24 21:59                 ` push.default??? 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=20090623103428.GA4214@pvv.org \
    --to=finnag@pvv.org \
    --cc=bonzini@gnu.org \
    --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 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.