Hallo Bj=C3=B6rn,

Bj=C3=B6rn Steinbrink schrieb am Sat 22. Mar, 16:12 (+0100):
quoted text
> On 2008.03.22 15:06:48 +0100, J=C3=B6rg Sommer wrote:
> > Bj=C3=B6rn Steinbrink schrieb am Sat 22. Mar, 02:52 (+0100):
> > > On 2008.03.22 02:19:42 +0100, J=C3=B6rg Sommer wrote:
> > > > The current version of git-rebase--interactive shows the user the c=
ommits
quoted text
> > > > coming from a merge.
> > > >=20
> > > > M---A---B
> > > >  \       \
> > > >   o---o---+---o branch
> > > >=20
> > > > Rebasing branch on M with preserve merges gives the commits A and B=
=2E But
quoted text
> > > > if you mark them for editing or remove them the rebase fails. You m=
ust
quoted text
> > > > keep them as they are. It's useless to bother the user with these c=
ommits
quoted text
> > > > and might lead to mistakes.
> > >=20
> > > Uhm, why do you completely remove the possibility to edit A
> >=20
> > Ahh, now I see what you've tried to say. I did add the option
> > --first-parent for rebase interactive *without* preserve merges, too.
> > I'll update my patch.
>=20
> I didn't even look at it closely enough to notice that.
> --preserve-merges preserves the structure of the history. You seem to
> interpret it as to preserve the merges against the original parents,
> except for the first one,

Yes, exactly this is my intent.

quoted text
> and that's simply not what it's meant to do.

That's a pity. So it's meant to be for such cases:

M---A---B
 \       \
  o---C---+---o branch

M---A---B
|    \
|     `-B'
 \       \
  o---C'--+---o branch

quoted text
> I can see how that might be useful, but you'd have to add that as an
> additional mode of operation, and not break the normal one.

What's the intention of the patch that adds --first-parent somewhere that
you've mentioned in the IRC?

I would like to send some tests for bugs I've seen. How do I correctly
cleanup after rebase failed? It's necessary to not break following tests.

test_expect_failure '=E2=80=A6' '
	=E2=80=A6
	git rebase -i =E2=80=A6
'

Should I do something like this:

test_expect_failure '=E2=80=A6' '
	=E2=80=A6
	if !git rebase -i =E2=80=A6;
	then
		git rebase --abort;
		false
	fi
'

Bye, J=C3=B6rg.
--=20
Manchmal denke ich, das sicherste Indiz daf=C3=BCr, da=C3=9F anderswo im Un=
iversum
intelligentes Leben existiert, ist, da=C3=9F niemand versucht hat, mit uns
Kontakt aufzunehmen.                           (Calvin und Hobbes)