BUG Commit Message

[Andreas Schneider]

On Friday 12 July 2013 13:09:30 Stefan  Metzmacher wrote:
> Hi Andreas,
> 
> > could we try to use the same scheme in the commit message for fixing bugs?
> > 
> > 	BUG XXXX: Short comment
> 
> I think the but number is not the most important thing to know about a
> commit.
> 
> e.g.
> 
> BUG 9994: Do not delete an existing valid credential cache.
> 
> says nothing without looking at bugzilla or the diff.
> 
> I think it's much more important to begin the line with the subsystem
> of the change (that's what I typically use and it's also what the linux
> kernel seems to use).
> 
> It's nice to have a bug number as additional information in the first line,
> but I'm not sure I like it to be the first thing I notice.
> 
> I typically use something like this:
> 
> s3:librpc: add support for PFC_FLAG_OBJECT_UUID when parsing packets
> (bug #9382)

I would also be fine with:

    smbd: Fix a 100% loop at shutdown time
    
    In the destructor of fsp->aio_requests[0] we put another request into
    fsp->aio_requests[0]. Don't overwrite that with TALLOC_FREE.
    
    BUG #12345

    Signed-off-by: Volker Lendecke <vl at samba.org>
    Reviewed-by: Jeremy Allison <jra at samba.org>


So that it comes after the long description. I would prefer upper case cause 
then it is easier to search for. A text could say: Fix a bug in foo.


	-- andreas

-- 
Andreas Schneider                   GPG-ID: F33E3FC6
Samba Team                             asn at samba.org
www.samba.org