[PATCH for comments] configure: install a whitespace checking pre-commit hook for developers

[Ralph Böhme]

On Tue, Apr 10, 2018 at 01:09:38PM +0200, Swen Schillig wrote:
> On Tue, 2018-04-10 at 12:50 +0200, Ralph Böhme via samba-technical
> wrote:
> > On Tue, Apr 10, 2018 at 11:36:04AM +0200, Swen Schillig wrote:
> > > I promise I will mention it only this one time again, but what
> > > about using
> > > checkpatch instead of reinventing the wheel ?
> > > 
> > > checkpatch is used in a variety of other non-kernel projects,
> > > like GANESHA (user space NFS server) which is a project with 
> > > very similar requirements.
> > > 
> > > So why not having a look at least ?
> > 
> > heavens, that's 6k+ lines of Perl and a lot of it is really kernel
> > specific.
> > 
> True, but is it not better to have matured 6k LOC which 
> we don't have to maintain vs 500 LOC where we do ?

I guess in the end we will have to maintain a copy as well. I haven't looked in
detail, but I guess we will need changes to checkpatch as well. So it means
maintaining a few hundred clean Python lines of a few thousand Perl.

-slow

-- 
Ralph Boehme, Samba Team       https://samba.org/
Samba Developer, SerNet GmbH   https://sernet.de/en/samba/
GPG Key Fingerprint:           FAE2 C608 8A24 2520 51C5
                               59E4 AA1E 9B71 2639 9E46