[PATCH for comments] configure: install a whitespace checking pre-commit hook for developers
Martin Schwenke
martin at meltin.net
Tue Apr 10 11:32:24 UTC 2018
On Tue, 10 Apr 2018 11:58:18 +0200, Stefan Metzmacher via
samba-technical <samba-technical at lists.samba.org> wrote:
> The logic for using a script that is part of the checkout and having a
> .git/hook/pre-commit script that just calls into the working tree
> is what I'd like to have (and maybe the logic to detect an outdated script)
I think this is a good approach. This allows us to refine the tool or
even a choice/combination of tools. There is a question about how to do
this neatly...
The "git commit --no-verify" option to avoid the pre-commit hook can
even be easily done when committing using magit under Emacs (and perhaps
also with the git support for vim).
Hmmm... that said, Douglas' pre-commit hook doesn't work under magit.
It works if I don't specify any paths. For some reason it doesn't
trigger if I do add the paths that match C and/or Python files. I
think there's some subtlety in the way the paths are specified in the
environment magit is running git in. :-(
Any other magit users on the team who might have a clue about this?
peace & happiness,
martin
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 833 bytes
Desc: OpenPGP digital signature
URL: <http://lists.samba.org/pipermail/samba-technical/attachments/20180410/78ef771c/attachment.sig>
More information about the samba-technical
mailing list