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
next prev parent 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