git.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
From: "Santi Béjar" <sbejar@gmail.com>
To: "Bill Lear" <rael@zopyra.com>
Cc: "Jeff King" <peff@peff.net>, "Jakub Narebski" <jnareb@gmail.com>,
	git@vger.kernel.org
Subject: Re: Dangers of working on a tracking branch
Date: Fri, 16 Feb 2007 17:10:57 +0100	[thread overview]
Message-ID: <8aa486160702160810i3d92ba6fy390674dfcc9c584@mail.gmail.com> (raw)
In-Reply-To: <17877.52593.509791.685008@lisa.zopyra.com>

On 2/16/07, Bill Lear <rael@zopyra.com> wrote:
> On Friday, February 16, 2007 at 10:21:30 (-0500) Jeff King writes:
> >On Fri, Feb 16, 2007 at 09:13:22AM -0600, Bill Lear wrote:
> >
> >> % git pull ../peer_repo topic:topic
> >> [...]
> >> * refs/heads/topic: not updating to non-fast forward branch 'topic' of ../peer_repo
> >> [...]
> >> So, why does it say "not updating to non-fast forward branch", yet
> >> it does the merge and gets the changes anyway?
> >
> >Because your pull command really means "merge in the topic branch from
> >peer_repo, and while you're at it, store it in my local tracking branch
> >topic". Remember that pull is really a fetch+merge. But the fetch is
> >actually doing _two_ things: putting the fetched branch into FETCH_HEAD,
> >and putting it in into refs/heads/topic. The latter fails (because of a
> >non-fastforward), but pull actually uses the FETCH_HEAD results to
> >do the merge.
> >
> >Yes, this seems overly complex for what you're doing, but the reason for
> >FETCH_HEAD is to support pulls when you _don't_ have a tracking branch
> >at all (i.e., 'git pull ../peer_repo topic').
>
> Ok, fair enough, but then I guess I'm back to my original question:
> how can I give a concrete demonstration to our developers that this is
> a bad thing?
>

It no longer works with recent git, as of v1.4.4.1-37-gd25c26e. Now
git-fetch exit with a non-zero status when fast-forward check fails,
so the merge does not happen.

Santi

  parent reply	other threads:[~2007-02-16 16:11 UTC|newest]

Thread overview: 22+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2007-02-15 20:49 Dangers of working on a tracking branch Bill Lear
2007-02-15 21:00 ` Nicolas Pitre
2007-02-15 21:21   ` Bill Lear
2007-02-15 21:33     ` Bill Lear
2007-02-15 21:43       ` Jeff King
2007-02-15 21:53         ` Bill Lear
2007-02-15 21:58           ` Jeff King
2007-02-15 22:04             ` Bill Lear
2007-02-15 22:09               ` Shawn O. Pearce
2007-02-15 22:40                 ` Bill Lear
2007-02-15 22:49                   ` Shawn O. Pearce
2007-02-15 22:06             ` Jeff King
2007-02-15 22:30               ` Bill Lear
2007-02-16  2:00       ` Jakub Narebski
2007-02-16 15:13         ` Bill Lear
2007-02-16 15:21           ` Jeff King
2007-02-16 15:27             ` Bill Lear
2007-02-16 15:52               ` Jeff King
2007-02-16 16:10               ` Santi Béjar [this message]
2007-02-16 16:34           ` Nicolas Pitre
2007-02-16 16:40             ` Bill Lear
2007-02-15 22:14     ` Nicolas Pitre

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=8aa486160702160810i3d92ba6fy390674dfcc9c584@mail.gmail.com \
    --to=sbejar@gmail.com \
    --cc=git@vger.kernel.org \
    --cc=jnareb@gmail.com \
    --cc=peff@peff.net \
    --cc=rael@zopyra.com \
    /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).