I am sorry but consensus among whom? There were several long discussions on the developers list about trailing whitespaces and no consensus was reached in any of them. If there is a strong consensus among VTK developers, that's fine but honestly this would be at the bottom of my priority list.<div>
<br></div><div>-berk<br><br><div class="gmail_quote">On Thu, Feb 9, 2012 at 11:57 AM, 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">
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>
<div class="im HOEnZb"><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>> 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>
</div><div class="im HOEnZb">--<br>
Unpaid intern in BillsBasement at noware dot com<br>
</div><div class="HOEnZb"><div class="h5">_______________________________________________<br>
Powered by <a href="http://www.kitware.com" target="_blank">www.kitware.com</a><br>
<br>
Visit other Kitware open-source projects at <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>
</div></div></blockquote></div><br></div>