From: Michal Marek <mmarek@suse.cz>
To: Linus Torvalds <torvalds@linux-foundation.org>
Cc: linux-kernel@vger.kernel.org, linux-kbuild@vger.kernel.org,
a.beregalov@gmail.com, andi@firstfloor.org, dzickus@redhat.com,
fejes@joco.name, g.liakhovetski@gmx.de, gthelen@google.com,
hschauhan@nulltrace.org, hui.zhu@windriver.com,
jan3sobi3ski@gmail.com, jay@goldhive.com,
jgunthorpe@obsidianresearch.com, jkacur@redhat.com,
joe@perches.com, kirr@mns.spb.ru, lizf@cn.fujitsu.com,
nir.tzachar@gmail.com, rabin@rab.in, rientjes@google.com,
roland@redhat.com, saalwaechter@gmail.com, shemminger@vyatta.com,
tabbott@ksplice.com, u.kleine-koenig@pengutronix.de,
vbendeb@google.com, vda.linux@googlemail.com,
wuzhangjin@gmail.com, xt28@gmx.de
Subject: Re: [GIT] kbuild changes for 2.6.35
Date: Tue, 01 Jun 2010 21:55:24 +0200 [thread overview]
Message-ID: <4C0565AC.8020205@suse.cz> (raw)
In-Reply-To: <alpine.LFD.2.00.1006010823080.3637@i5.linux-foundation.org>
On 1.6.2010 17:32, Linus Torvalds wrote:
> No, the commits themselves are likely fine, although for the future it
> really would be good to make things like that more descriptive. I just
> want people to try to argue for _why_ I should do a pull, and _what_ I'm
> getting in their "please pull" thing.
>
> It's not always necessary, and some people do it better than others. For
> an example of a really good pull request, look at the ones David Miller
> sends me for networking - they explain what's going on in the pull, so
> it's always easy to pull them because just the request makes me feel like
> David is really on top of things, and lets me have some 30'000 ft overview
> of what's going on.
OK, I'll try to do better job next time. I'm also going to use different
branches for kbuild / kconfig / trivial stuff like .gitignore / etc from
now on, so that you don't get a all-or-nothing pull request in the next
merge window (something that Sam suggested).
> At the same time, in many cases I obviously pull _without_ any kind of
> real explanation - and that tends to be especially true with maintainers
> that I've worked with for a long time, or areas that are so specialized
> that they are almost self-explanatory (let's be honest: when a filesystem
> maintainer asks me to pull their special filesystem, I'm perfectly happy
> with the overview of "30 changesets to XFS", and there's no need for much
> explanation, although a rough overview of what's been going on is always
> good to see).
>
> So the reason I ask for explanations for kbuild is that not only have we
> had different maintainers, it's an area that affects a lot of different
> things and has historically had issues with odd architectures or old
> binutils tools etc.
I see. Thanks a lot for pulling the changes now.
Michal
prev parent reply other threads:[~2010-06-01 19:55 UTC|newest]
Thread overview: 3+ messages / expand[flat|nested] mbox.gz Atom feed top
[not found] <20100527144459.GA14513@sepie.suse.cz>
[not found] ` <alpine.LFD.2.00.1005300921000.4639@i5.linux-foundation.org>
2010-05-31 12:02 ` [GIT] kbuild changes for 2.6.35 Michal Marek
2010-06-01 15:32 ` Linus Torvalds
2010-06-01 19:55 ` Michal Marek [this message]
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=4C0565AC.8020205@suse.cz \
--to=mmarek@suse.cz \
--cc=a.beregalov@gmail.com \
--cc=andi@firstfloor.org \
--cc=dzickus@redhat.com \
--cc=fejes@joco.name \
--cc=g.liakhovetski@gmx.de \
--cc=gthelen@google.com \
--cc=hschauhan@nulltrace.org \
--cc=hui.zhu@windriver.com \
--cc=jan3sobi3ski@gmail.com \
--cc=jay@goldhive.com \
--cc=jgunthorpe@obsidianresearch.com \
--cc=jkacur@redhat.com \
--cc=joe@perches.com \
--cc=kirr@mns.spb.ru \
--cc=linux-kbuild@vger.kernel.org \
--cc=linux-kernel@vger.kernel.org \
--cc=lizf@cn.fujitsu.com \
--cc=nir.tzachar@gmail.com \
--cc=rabin@rab.in \
--cc=rientjes@google.com \
--cc=roland@redhat.com \
--cc=saalwaechter@gmail.com \
--cc=shemminger@vyatta.com \
--cc=tabbott@ksplice.com \
--cc=torvalds@linux-foundation.org \
--cc=u.kleine-koenig@pengutronix.de \
--cc=vbendeb@google.com \
--cc=vda.linux@googlemail.com \
--cc=wuzhangjin@gmail.com \
--cc=xt28@gmx.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