From: Jonathan Nieder <jrnieder@gmail.com>
To: Michael J Gruber <git@drmicha.warpmail.net>
Cc: Junio C Hamano <gitster@pobox.com>,
Ramkumar Ramachandra <artagnon@gmail.com>,
Michael Schubert <mschub@elegosoft.com>,
Git List <git@vger.kernel.org>, Jeff King <peff@peff.net>
Subject: Re: Proposal: branch.<name>.remotepush
Date: Fri, 8 Feb 2013 10:36:29 -0800 [thread overview]
Message-ID: <20130208183629.GA8461@google.com> (raw)
In-Reply-To: <5114D5B0.5080906@drmicha.warpmail.net>
Michael J Gruber wrote:
> Junio C Hamano venit, vidit, dixit 08.02.2013 09:16:
>> Jonathan Nieder <jrnieder@gmail.com> writes:
>>> "Wait, why did the remote rewind?"
>>
>> Oh, I am very well aware of that glitch.
>>
>> "git push" has this hack to pretend as if the pusher immediately
>> turned around and fetched from the remote.
>>
>> It shouldn't have been made to do so unconditionally; instead it
>> should have been designed to give the pushee a way to optionally
>> tell you "I acccept this push, but you may not see it to be updated
>> to that exact value you pushed when you fetched from me right now".
Yes, I agree with this.
The "git push" hack does seem to be useful in practice for helping
people just starting to use git. If they have a separate "gitk --all"
window open, they can refresh it and see the remote-tracking branch
corresponding to the branch that has been pushed advancing. It matches
a model in which remote-tracking refs represent "git's idea of where
these branches are in the remote repository".
And in that model, a remote being able to respond to a push with
"ref update queued, but please keep in mind that it may take me a
while to chew through that queue" should be perfectly reasonable.
[...]
> And this seems to be more natural, too. It can keep the internals (the
> auxiliary ref on the server side) hidden from the user.
Just to clarify: this is not an internal ref being exposed. No
auxiliary refs/for/master ref actually exists. The ref Gerrit users
push to is a UI fiction.
That's important because otherwise two developers could not propose
changes for the same branch at the same time.
Jonathan
next prev parent reply other threads:[~2013-02-08 18:37 UTC|newest]
Thread overview: 25+ messages / expand[flat|nested] mbox.gz Atom feed top
2013-02-07 16:14 Proposal: branch.<name>.remotepush Ramkumar Ramachandra
2013-02-07 17:45 ` Michael Schubert
2013-02-07 19:37 ` Ramkumar Ramachandra
2013-02-07 19:49 ` Ramkumar Ramachandra
2013-02-07 20:14 ` Ramkumar Ramachandra
2013-02-07 23:30 ` Jonathan Nieder
2013-02-07 23:41 ` Junio C Hamano
2013-02-08 7:48 ` Jonathan Nieder
2013-02-08 8:16 ` Junio C Hamano
2013-02-08 10:38 ` Michael J Gruber
2013-02-08 17:11 ` Junio C Hamano
2013-02-08 17:16 ` Junio C Hamano
2013-02-08 18:29 ` Ramkumar Ramachandra
2013-02-08 19:13 ` Junio C Hamano
2013-02-08 18:36 ` Jonathan Nieder [this message]
2013-02-08 18:42 ` Ramkumar Ramachandra
2013-02-08 19:18 ` Junio C Hamano
2013-02-09 7:29 ` Ramkumar Ramachandra
2013-02-08 4:48 ` Jeff King
2013-02-08 6:08 ` Junio C Hamano
2013-02-08 6:28 ` Jeff King
2013-02-08 6:45 ` Junio C Hamano
2013-02-08 9:22 ` Jeff King
2013-02-08 17:06 ` Junio C Hamano
2013-02-08 18:03 ` Ramkumar Ramachandra
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=20130208183629.GA8461@google.com \
--to=jrnieder@gmail.com \
--cc=artagnon@gmail.com \
--cc=git@drmicha.warpmail.net \
--cc=git@vger.kernel.org \
--cc=gitster@pobox.com \
--cc=mschub@elegosoft.com \
--cc=peff@peff.net \
/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.