Bug#691389: [checkbashisms] multiple bug fixes and new checks

James McCoy jamessan at debian.org
Fri Oct 26 16:13:37 UTC 2012


On Fri, Oct 26, 2012 at 02:31:34PM +0200, Benjamin Drung wrote:
> Am Donnerstag, den 25.10.2012, 17:33 -0500 schrieb Raphael Geissert:
> > > Please do not sign off your own patches. Your patches will be signed off
> > > by the person that applies your patches.
> > 
> > Perhaps we differ on what the sign off means, and that shouldn't cause any 
> > issue for another person to sign off them as well.
> 
> I found two meaning of signing off patches:
> 
> 1) The signer certifies that he/she have created the patch in question
> and take responsibility for the copyright status of the code in
> question.
> 
> 2) The signer reviewed the patch and has done some sort of QA.
> 
> We use the second meaning.

We do, along with an optional application of the first.  I tend to do
both, but not everyone does.  I see no reason to request people elide
the first type.

-- 
James
GPG Key: 4096R/331BA3DB 2011-12-05 James McCoy <jamessan at debian.org>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 836 bytes
Desc: Digital signature
URL: <http://lists.alioth.debian.org/pipermail/devscripts-devel/attachments/20121026/183e56a3/attachment.pgp>


More information about the devscripts-devel mailing list