Whitespace and bullying

Swen Schillig swen at vnet.ibm.com
Mon Apr 9 13:01:02 UTC 2018


On Mon, 2018-04-09 at 22:41 +1000, Martin Schwenke wrote:
> On Mon, 09 Apr 2018 14:28:45 +0200, Swen Schillig <swen at vnet.ibm.com>
> wrote:
> 
> > On Mon, 2018-04-09 at 21:11 +1000, Martin Schwenke via samba-
> > technical
> > wrote:
> > > Swen's suggestion of using the kernel's checkpatch is reasonable,
> > > but
> > > the code will require modifications.  It doesn't cope with more
> > > than
> > > one file in a patch,   
> > No, what makes you think so ?
> 
> Sorry, typo.  I meant more than 1 commit in a patch file.  It barfs
> because it thinks the Signed-off-by: tags for most of the commits are
> duplicates.
True.
But that should be configured/"hard coded" in the git config anyway :-)

> 
> Yes, it can be worked around...  but I think the kernel checkpatch.pl
> makes too many kernel-related assumptions...
Again true, but that's what a checkpatch.conf could be for, 
maybe created by Jeremy and Volker, just to make sure :-)

There are already a few versions out there for projects other than the
kernel. I guess Ronnie mentioned that already.

Cheers Swen.




More information about the samba-technical mailing list