[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