Git development
 help / color / mirror / Atom feed
From: Linus Torvalds <torvalds@linux-foundation.org>
To: Steve French <smfrench@gmail.com>
Cc: git@vger.kernel.org
Subject: Re: CIFS fixes
Date: Sun, 11 May 2008 10:04:52 -0700 (PDT)	[thread overview]
Message-ID: <alpine.LFD.1.10.0805110957530.3330@woody.linux-foundation.org> (raw)
In-Reply-To: <524f69650805110953t6561ff84t1867681bea093d35@mail.gmail.com>



On Sun, 11 May 2008, Steve French wrote:
>
> That is not what I meant.   I meant that since May 6th I only did one
> - and those messages still showed up.  So I just ran a git-rebase
> origin which removed them

You're doing something wrong then, and your rebased result is suspect.

Have you done a "git fetch" to fetch what is in my current tree? Because 
if you haven't, then you're generating the "this is the new state" without 
actually taking into account that the old state was already pulled!

And that *old* state contains those four merges that I got from your 
previous pull request!

So now you likely rebased the commits that I already merged (again, 
because you *think* they are just local to your branch, because you 
haven't updated your origin reference point), and they are now duplicates 
of something I already have (but with different commit ID's, since your 
rebase has moved them around in the history).

So now, if I were to pull again, I'd just get the same commits all over 
again, just as duplicates (plus any new ones, of course). Git would 
probably merge it all fine - unless your new ones were to the same area as 
the old ons, in which case it might be unhappy about the fact that both 
branches changed things in the same area - but the history would be crud.

In other words: you *must*not* rebase stuff that you have already 
publicized. That just creates problems.

The good news is that you can most likely fix it all up by just doing

	git fetch
	git rebase origin

because now the *new* rebase will try to rebase it all over again, but now 
it will see that I already merged the old ones, so the rebase will just 
skip those commits, and you should have only the *real* new ones pending 
again.

		Linus

  reply	other threads:[~2008-05-11 17:05 UTC|newest]

Thread overview: 6+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
     [not found] <524f69650805082054g43823f85i623cb2c11cd01039@mail.gmail.com>
     [not found] ` <alpine.LFD.1.10.0805090810390.3142@woody.linux-foundation.org>
2008-05-11 16:42   ` CIFS fixes Steve French
2008-05-11 16:52     ` Linus Torvalds
2008-05-11 16:53       ` Steve French
2008-05-11 17:04         ` Linus Torvalds [this message]
2008-05-11 17:48           ` Steve French
2008-05-11 19:20             ` Linus Torvalds

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=alpine.LFD.1.10.0805110957530.3330@woody.linux-foundation.org \
    --to=torvalds@linux-foundation.org \
    --cc=git@vger.kernel.org \
    --cc=smfrench@gmail.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