[alsa-devel] rfc: rewrite commit subject line for subsystem maintainer preference tool

Randy Dunlap rdunlap at xenotime.net
Tue Nov 16 21:21:02 CET 2010


On Tue, 16 Nov 2010 19:55:31 +0000 Mark Brown wrote:

> On Tue, Nov 16, 2010 at 08:35:22PM +0100, Florian Mickler wrote:
> 
> > Hehe, I know that I wouldn't want to hand edit every autogenerated patch
> > people throw at me... What about just dropping everything before the
> > last "]" or ":" and putting an autogenerated prefix before it in a
> > pre-commit hook on your side?  
> 
> > That should work most of the time... don't know... maybe other
> 
> It's the most of the time bit that worries me, I'm generally reluctant
> to script things like this when the scripts aren't very widely used and
> it's a pain to get hooks distributed over all my systems and working for
> all the things I need to apply patches for.
> 
> From my point of view my current approach is actually working pretty
> well with most submitters, even people doing similar janitorial stuff.

I don't know what you asked Joe to change, but asking someone to use
the documented canonical patch format:

<quote>
The canonical patch subject line is:

    Subject: [PATCH 001/123] subsystem: summary phrase
</quote>

should be fine.  And there is no need for printf-ish templates
for this in MAINTAINERS either.

---
~Randy
*** Remember to use Documentation/SubmitChecklist when testing your code ***


More information about the Alsa-devel mailing list