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

Jeremy Allison jra at samba.org
Wed Apr 25 15:42:25 UTC 2018


On Wed, Apr 25, 2018 at 06:40:48PM +1000, Martin Schwenke via samba-technical wrote:
> On Wed, 25 Apr 2018 08:03:49 +0200, Ralph Böhme <slow at samba.org> wrote:
> 
> > On Wed, Apr 18, 2018 at 09:14:07AM +1000, Martin Schwenke via samba-technical wrote:
> > > On Wed, 11 Apr 2018 11:59:45 +0200, Ralph Böhme via samba-technical
> > > <samba-technical at lists.samba.org> wrote:  
> > > > On Wed, Apr 11, 2018 at 10:04:32AM +0200, Stefan Metzmacher wrote:  
> >  [...]  
> > > > 
> > > > Added to both hook and script.
> > > > 
> > > > Updated patch attached, please push if happy. Thanks!  
> > > 
> > > I notice this hasn't gone in yet.  Metze, are you happy with it?  
> > 
> > also, how does/will this align with what Simo is proposing?
> 
> I think this should go in now to avoid the problem we were actually
> seeing.
> 
> I think Simo's tool should go in too so that we can play with it.  I've
> already used it to clean up some of my commits.  Those who have enough
> courage can add it to their commit hook (possibly ignoring the result
> and using it to provide warnings).  Eventually we can even look at
> adding it to tests.
> 
> However, I think we need something simple now, while we work on the
> long term solution and long term approach.

Yes, I agree. What is the blocker on this immediate fix ? Metze,
are you OK with this ? If not can you articulate the problem ?



More information about the samba-technical mailing list