[PATCH for comments] configure: install a whitespace checking pre-commit hook for developers
Martin Schwenke
martin at meltin.net
Wed Apr 11 02:10:18 UTC 2018
On Tue, 10 Apr 2018 14:45:24 +0200, Ralph Böhme <slow at samba.org> wrote:
> On Tue, Apr 10, 2018 at 02:38:36PM +0200, Ralph Böhme wrote:
> > On Tue, Apr 10, 2018 at 02:29:41PM +0200, Swen Schillig wrote:
> > > Ralph, don't you need to add that "--cached" parameter to your script
> > > as well ?
> > > As stated in one of my earlier mails, I believe you need that 'cause
> > > otherwise all modified files are checked and not just the ones which
> > > are part of the commit (were add'ed before).
> >
> > are you sure? It's a commit hook after all so my expectation was it only check
> > what I commit. I'll check...
>
> d'oh! You're obviously right...
>
> Updated patch attached.
Looks good:
Reviewed-by: Martin Schwenke <martin at meltin.net>
We can followup with a patch to make it work under magit, probbaly via:
unset GIT_LITERAL_PATHSPECS
Alternatively, if others agree then this can go in now.
peace & happiness,
martin
More information about the samba-technical
mailing list