BUG Commit Message
Andreas Schneider
asn at samba.org
Fri Jul 12 05:25:01 MDT 2013
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
More information about the samba-technical
mailing list