From: Linus Torvalds <torvalds@linux-foundation.org>
To: David Miller <davem@davemloft.net>
Cc: tglx@linutronix.de, linux-kernel@vger.kernel.org, mingo@elte.hu,
hpa@zytor.com
Subject: Re: [GIT pull] x86 fixes for 2.6.26
Date: Fri, 16 May 2008 18:38:26 -0700 (PDT) [thread overview]
Message-ID: <alpine.LFD.1.10.0805161831380.3020@woody.linux-foundation.org> (raw)
In-Reply-To: <20080516.172835.260358594.davem@davemloft.net>
On Fri, 16 May 2008, David Miller wrote:
>
> I think I merged from Linus's tree only 3 or 4 times maximum since I
> created that tree way back when the 2.6.26 merge window opened up.
Basically, a rule-of-thumb would be "once a week is reasonable", but on
the other hand, if you have actual conflicts, more often is fine, and if
you know your area isn't impacted (eg most filesystems), you'd probably
only try to synchronize at release points or something like that.
The reason to avoid doing overly many merges is
- merging with me at random points is usually not a good idea anyway,
unless you have a really strong reason for it. Yes, my tree is fairly
stable, but still..
(This also implies that merging with my *tags* is usually a better idea
than merging with some random point in time, and is one reason it makes
sense to try to merge with major releases rather than anything else)
- the history just looks cleaner and is easier to follow when there
aren't criss-crossing merges.
This may not matter most of the time, but it *does* make a difference
when doing "git bisect". I don't know about others, but I often do "git
bisect visualize" then I have some totally unknown bug and I'm trying
to guess what's going on - it's a great way to give people a heads up
saying "ok, I'm in the middle of a bisection run, and it _looks_ like
it may be due to you".
So trying to have fairly clean history is worth it (it also makes it
slightly faster to bisect when you don't have lots of criss-cross merges,
but that's a fairly small factor).
> If users, or even you, want to get a bug fix from Linus's tree or check
> if there will be merge conflicts, just create a test branch and play
> with such things there.
Yes. I think you guys already have a test branch for the "join it all
together" case, don't you?
Linus
next prev parent reply other threads:[~2008-05-17 1:39 UTC|newest]
Thread overview: 24+ messages / expand[flat|nested] mbox.gz Atom feed top
2008-05-16 22:38 Thomas Gleixner
2008-05-16 22:47 ` Linus Torvalds
2008-05-16 22:51 ` Linus Torvalds
2008-05-16 23:44 ` Thomas Gleixner
2008-05-17 0:03 ` Linus Torvalds
2008-05-17 0:28 ` David Miller
2008-05-17 1:38 ` Linus Torvalds [this message]
2008-05-17 19:39 ` Ingo Molnar
2008-05-17 20:00 ` Linus Torvalds
2008-05-17 21:02 ` Thomas Gleixner
2008-05-17 21:36 ` Linus Torvalds
2008-05-17 1:57 ` Theodore Tso
2008-05-17 3:19 ` Linus Torvalds
2008-05-17 14:58 ` Theodore Tso
2008-05-17 17:05 ` Linus Torvalds
2008-05-17 20:37 ` Thomas Gleixner
2008-05-17 20:26 ` Junio C Hamano
2008-05-17 22:45 ` Jesper Juhl
2008-05-18 0:35 ` Linus Torvalds
2008-05-18 2:22 ` Stephen Rothwell
2008-05-18 22:09 ` Jesper Juhl
2008-05-18 22:26 ` Linus Torvalds
2008-05-20 0:01 ` Jesper Juhl
2008-05-23 21:49 Thomas Gleixner
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.0805161831380.3020@woody.linux-foundation.org \
--to=torvalds@linux-foundation.org \
--cc=davem@davemloft.net \
--cc=hpa@zytor.com \
--cc=linux-kernel@vger.kernel.org \
--cc=mingo@elte.hu \
--cc=tglx@linutronix.de \
/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
all inboxes | Powered by JetHome®