[<prev] [next>] [<thread-prev] [day] [month] [year] [list]
Message-ID: <alpine.LNX.2.00.1010242334330.6879@pobox.suse.cz>
Date: Sun, 24 Oct 2010 23:37:16 +0200 (CEST)
From: Jiri Kosina <jkosina@...e.cz>
To: Linus Torvalds <torvalds@...ux-foundation.org>
Cc: linux-kernel@...r.kernel.org
Subject: Re: [GIT] trivial
On Sun, 24 Oct 2010, Linus Torvalds wrote:
> > please pull from 'for-next' branch at
> >
> > git://git.kernel.org/pub/scm/linux/kernel/git/jikos/trivial.git for-next
>
> Pulled. With one modification: I didn't take the top merge commit.
>
> I _really_ prefer to see the clashes myself, and the extra merges just
> make history less readable. So if the conflicts are trivial, please
> don't pre-merge stuff - it just hides things from me.
>
> Now, admittedly the things it hides shouldn't matter for something
> like the trivial tree, but at the same time, exactly because it's the
> trivial tree I would also expect the conflicts to be really trivial,
> so I'd rather just have people follow the general rule of "please
> don't pre-merge" regardless.
>
> It's a good idea to perhaps do a private merge just to see how nasty
> things are, and to perhaps give me a heads-up (and maybe even export
> the pre-merged thing as a "btw, if you're lazy and don't care about
> the conflicts, you can pull this other pre-merged branch" branch). But
> on the whole, I tend to find the conflicts to be the most telling
> parts of any merge, and be a good hint about what development trees
> clashed with each other.
OK, I always do private merge with your latest tree to see what actually
happens, and only merge with latest state of your tree whenever there is
conflict. But deferring this to you (with warning in the pull request) is
obviously fine with me as well.
--
Jiri Kosina
SUSE Labs, Novell Inc.
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
Powered by blists - more mailing lists