From: Johannes Schindelin <Johannes.Schindelin@gmx.de>
To: Benoit Sigoure <tsuna@lrde.epita.fr>
Cc: Steffen Prohaska <prohaska@zib.de>,
Johannes Sixt <j.sixt@viscovery.net>,
Andreas Ericsson <ae@op5.se>,
Ralf Wildenhues <Ralf.Wildenhues@gmx.de>,
Git Mailing List <git@vger.kernel.org>
Subject: Re: [PATCH v2] user-manual: add advanced topic "bisecting merges"
Date: Thu, 8 Nov 2007 15:23:15 +0000 (GMT) [thread overview]
Message-ID: <Pine.LNX.4.64.0711081520170.4362@racer.site> (raw)
In-Reply-To: <5498B368-053A-424E-B901-A55B66FF5801@lrde.epita.fr>
Hi,
On Thu, 8 Nov 2007, Benoit Sigoure wrote:
> On Nov 8, 2007, at 1:54 PM, Steffen Prohaska wrote:
>
> > Do you use rebase like this in real life?
> >
> > I thought of the text as background information that might be helpful
> > for users who want do decide wether to merge or to rebase. The problem
> > described may be valuable information supporting a decision about a
> > recommended workflow for a group of users.
>
> You're missing the point. Johannes suggested that you rebase *only* for
> bisecting purpose. Once you find the culprit commit, throw away your
> rebased stuff.
Just to clear things up: it was the other Johannes who suggested it.
But I strongly advise not to rebase before bisecting, since you could very
well end up changing the behaviour of the program by rebasing it. Even to
a point where you cannot bisect it any longer, for example when the merge
of two branches contains an important fix without which the combined
branches (or even parts of them) will not even compile.
Last time I checked, however, bisect worked like a charm even on
a history with complicated ancestry.
Ciao,
Dscho
next prev parent reply other threads:[~2007-11-08 15:23 UTC|newest]
Thread overview: 30+ messages / expand[flat|nested] mbox.gz Atom feed top
2007-11-04 9:16 [PATCH] user-manual: add advanced topic "bisecting merges" Steffen Prohaska
2007-11-04 11:23 ` Ralf Wildenhues
2007-11-07 21:50 ` [PATCH v2] " Steffen Prohaska
2007-11-07 22:16 ` Benoit Sigoure
2007-11-07 22:26 ` J. Bruce Fields
2007-11-08 6:40 ` Steffen Prohaska
2007-11-08 7:19 ` Johannes Sixt
2007-11-08 8:59 ` Steffen Prohaska
2007-11-08 9:11 ` Johannes Sixt
2007-11-08 9:33 ` Andreas Ericsson
2007-11-08 9:53 ` Johannes Sixt
2007-11-08 12:54 ` Steffen Prohaska
2007-11-08 13:22 ` Johannes Sixt
2007-11-08 14:55 ` Steffen Prohaska
2007-11-10 9:48 ` [PATCH v3] " Steffen Prohaska
2007-11-10 10:36 ` Junio C Hamano
2007-11-10 11:16 ` Steffen Prohaska
2007-11-10 13:49 ` [PATCH v4] user-manual: Add section "Why bisecting merge commits can be harder ..." Steffen Prohaska
2007-11-10 19:10 ` Linus Torvalds
2007-11-10 20:25 ` Junio C Hamano
2007-11-10 22:41 ` Steffen Prohaska
2007-11-18 3:59 ` J. Bruce Fields
2007-11-18 9:47 ` Steffen Prohaska
2007-11-18 23:18 ` J. Bruce Fields
2007-11-08 13:38 ` [PATCH v2] user-manual: add advanced topic "bisecting merges" Andreas Ericsson
2007-11-08 14:51 ` Benoit Sigoure
2007-11-08 15:07 ` Steffen Prohaska
2007-11-08 15:23 ` Johannes Schindelin [this message]
2007-11-08 18:38 ` Brian Gernhardt
2007-11-04 13:50 ` [PATCH] " Benoit SIGOURE
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=Pine.LNX.4.64.0711081520170.4362@racer.site \
--to=johannes.schindelin@gmx.de \
--cc=Ralf.Wildenhues@gmx.de \
--cc=ae@op5.se \
--cc=git@vger.kernel.org \
--cc=j.sixt@viscovery.net \
--cc=prohaska@zib.de \
--cc=tsuna@lrde.epita.fr \
/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).