public inbox for linux-kernel@vger.kernel.org
 help / color / mirror / Atom feed
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

      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