Messages in this thread | | | Date | Mon, 2 Mar 2009 10:34:37 -0800 | From | Andrew Morton <> | Subject | Re: [PATCH] checkpatch: Warn on empty commit log bodies |
| |
On Mon, 2 Mar 2009 18:24:57 +0000 Mark Brown <broonie@sirena.org.uk> wrote:
> On Mon, Mar 02, 2009 at 10:01:58AM -0800, Andrew Morton wrote: > > > I think what triggered this was a patch from Mark which had no > > changelog and which had me sitting there wondering wtf it does, whether > > we need it in 2.6.29, whether we need it in 2.6.28.x and earlier and me > > not having the foggiest clue then getting grumpy. > > Do you mean no changelog in the body of the e-mail here? I'm assuming > now that you mean no changelog in the body but when you say "no > changelog" that reads differently. I'm not saying the changelog was > perfect here but your comments really do read like you felt there was > nothing at all.
The text covering a patch should describe what the patch does, why it does it, how it does it and it should describe the end-user effects of not having the patch present. Any and all of these can be skipped if they are utterly obvious and unneeded.
Changes should be properly described, that's all. The means by which that is done isn't terribly important. Sometimes most of the description is in code comments, or in a newly-added Documentation/ file.
The reason I asked you personally to always send a changelog is because I quite frequently sit there scratching my head at your patches not having a clue what they do nor how to prioritise them.
| |