[PATCH for comments] configure: install a whitespace checking pre-commit hook for developers
metze at samba.org
Wed Apr 25 18:03:16 UTC 2018
Am 25.04.2018 um 17:42 schrieb Jeremy Allison via samba-technical:
> 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
>> 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 ?
-------------- next part --------------
A non-text attachment was scrubbed...
Size: 836 bytes
Desc: OpenPGP digital signature
More information about the samba-technical