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
next prev 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 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).