Re: [PATCH] pull: refuse complete src:dst fetchspec arguments
- From
Björn Steinbrink <b.steinbrink@gmx.de>
- Date
- Oct 21, 2009, 03:15 UTC
- Message-ID
- <20091021031528.GB18997@atjola.homenet>
- In-Reply-To
- <alpine.LNX.2.00.0910202044150.14365@iabervon.org>
On 2009.10.20 20:55:25 -0400, Daniel Barkalow wrote:
Show 8 quoted lines
> Maybe it should be fine to do: > > $ git fetch long-url-here master:temp > $ git merge temp > $ git checkout other-branch-that-also-needs-it > $ git merge temp > > But "temp" is "refs/remotes/temp", not "refs/heads/temp"?
One (maybe important) difference there is that the "pull" gets you:
Merge branch 'pu' of git://git.kernel.org/pub/scm/git/git
Even with "master:tmp". But with fetch+merge (storing in refs/remotes):
Merge remote branch 'tmp'
As a minor side-effect, having the "tmp" ref makes re-running the pull (for whatever reason) cheaper, as without it, the fetch step would possibly re-fetch the whole stuff (not reachable through any local ref).
Björn, undecided...