Brad and Marcus are going to investigate the possibility of automatically removing trailing spaces at git add or commit stage. If we do that, would anyone object to enforcing the no-trailing-whitespace rule as a commit hook and in Gerrit (in VTK 6)?<br>
<br><div class="gmail_quote">On Thu, Feb 9, 2012 at 12:11 PM, Bill Lorensen <span dir="ltr"><<a href="mailto:bill.lorensen@gmail.com">bill.lorensen@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
I should have said on gerrit. And going forward, we should definitely<br>
consider fixing it for VTK6.<br>
<div class="HOEnZb"><div class="h5"><br>
On Thu, Feb 9, 2012 at 12:06 PM, Berk Geveci <<a href="mailto:berk.geveci@kitware.com">berk.geveci@kitware.com</a>> wrote:<br>
> I am sorry but consensus among whom? There were several long discussions on<br>
> the developers list about trailing whitespaces and no consensus was reached<br>
> in any of them. If there is a strong consensus among VTK developers, that's<br>
> fine but honestly this would be at the bottom of my priority list.<br>
><br>
> -berk<br>
><br>
> On Thu, Feb 9, 2012 at 11:57 AM, Bill Lorensen <<a href="mailto:bill.lorensen@gmail.com">bill.lorensen@gmail.com</a>><br>
> wrote:<br>
>><br>
>> The consensus is to restore the client side hook for trailing white<br>
>> space. But rather than an error, it will be a warning. This will<br>
>> permit developers to avoid the warning on gerrit.<br>
>> See the gerrit topic: <a href="http://review.source.kitware.com/#/c/4219/" target="_blank">http://review.source.kitware.com/#/c/4219/</a><br>
>> For VTK6, the warning will become an error. The VTK6 team will run a<br>
>> script to cleanup existing code.<br>
>><br>
>><br>
>> On Tue, Feb 7, 2012 at 9:36 AM, Marcus D. Hanwell<br>
>> <<a href="mailto:marcus.hanwell@kitware.com">marcus.hanwell@kitware.com</a>> wrote:<br>
>> > On Mon, Feb 6, 2012 at 3:43 PM, Bill Lorensen <<a href="mailto:bill.lorensen@gmail.com">bill.lorensen@gmail.com</a>><br>
>> > wrote:<br>
>> >> Now that we are using gerrit for vtk changes, I think we need a<br>
>> >> trailing whitespace pre-commit hook. Currently this is caught by<br>
>> >> gerrit, but a developer does not see the defect on local commits.<br>
>> >><br>
>> > This mail gives a summary, we turned it off,<br>
>> ><br>
>> > <a href="http://www.vtk.org/pipermail/vtk-developers/2011-April/009638.html" target="_blank">http://www.vtk.org/pipermail/vtk-developers/2011-April/009638.html</a><br>
>> ><br>
>> > This is largely the reason for making it a warning in Gerrit (not an<br>
>> > error). I would like to turn it back on (by adjusting .gitattributes<br>
>> > in VTK) and leave it on as a warning in the Gerrit content check<br>
>> > robot. That way, if there is good reason, the --no-verify argument can<br>
>> > be used to bypass the local hooks.<br>
>> ><br>
>> > The complicating factor there is that this would also bypass the<br>
>> > Gerrit Change-Id hook, this is probably acceptable for those few cases<br>
>> > where you need to push through commits with whitespace issues but<br>
>> > perhaps too far for some? The checks only ever concerned new changes,<br>
>> > and would continue to do so.<br>
>> ><br>
>> > Marcus<br>
>><br>
>><br>
>><br>
>> --<br>
>> Unpaid intern in BillsBasement at noware dot com<br>
>> _______________________________________________<br>
>> Powered by <a href="http://www.kitware.com" target="_blank">www.kitware.com</a><br>
>><br>
>> Visit other Kitware open-source projects at<br>
>> <a href="http://www.kitware.com/opensource/opensource.html" target="_blank">http://www.kitware.com/opensource/opensource.html</a><br>
>><br>
>> Follow this link to subscribe/unsubscribe:<br>
>> <a href="http://www.vtk.org/mailman/listinfo/vtk-developers" target="_blank">http://www.vtk.org/mailman/listinfo/vtk-developers</a><br>
>><br>
><br>
<br>
<br>
<br>
--<br>
Unpaid intern in BillsBasement at noware dot com<br>
</div></div></blockquote></div><br>