All of lore.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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.