<div dir="ltr">+1 on the topic<div><br></div><div>How about we catch them in hacking so that they won't ever come back?</div></div><div class="gmail_extra"><br><br><div class="gmail_quote">On Thu, Oct 24, 2013 at 4:53 PM, Davanum Srinivas <span dir="ltr"><<a href="mailto:davanum@gmail.com" target="_blank">davanum@gmail.com</a>></span> wrote:<br>

<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">+1 to remove them.<br>
<br>
-- dims<br>
<div class="HOEnZb"><div class="h5"><br>
On Thu, Oct 24, 2013 at 8:44 AM, Monty Taylor <<a href="mailto:mordred@inaugust.com">mordred@inaugust.com</a>> wrote:<br>
><br>
><br>
> On 10/24/2013 08:38 AM, Joe Gordon wrote:<br>
>> Since the beginning of OpenStack we have had vim modelines all over the<br>
>> codebase, but after seeing this<br>
>> patch <a href="https://review.opeenstack.org/#/c/50891/" target="_blank">https://review.opeenstack.org/#/c/50891/</a><br>
>> <<a href="https://review.openstack.org/#/c/50891/" target="_blank">https://review.openstack.org/#/c/50891/</a>> I took a further look into vim<br>
>> modelines and think we should remove them. Before going any further, I<br>
>> should point out these lines don't bother me too much but I figured if<br>
>> we could get consensus, then we could shrink our codebase by a little bit.<br>
>><br>
>> Sidenote: This discussion is being moved to the mailing list because it<br>
>> 'would be better to have a mailing list thread about this rather than<br>
>> bits and pieces of discussion in gerrit' as this change requires<br>
>> multiple patches.  <a href="https://review.openstack.org/#/c/51295/" target="_blank">https://review.openstack.org/#/c/51295/</a>.<br>
>><br>
>><br>
>> Why remove them?<br>
>><br>
>> * Modelines aren't supported by default in debian or ubuntu due to<br>
>> security reasons: <a href="https://wiki.python.org/moin/Vim" target="_blank">https://wiki.python.org/moin/Vim</a><br>
>> * Having modelines for vim means if someone wants we should support<br>
>> modelines for emacs<br>
>> (<a href="http://www.gnu.org/software/emacs/manual/html_mono/emacs.html#Specifying-File-Variables" target="_blank">http://www.gnu.org/software/emacs/manual/html_mono/emacs.html#Specifying-File-Variables</a>)<br>


>> etc. as well.  And having a bunch of headers for different editors in<br>
>> each file seems like extra overhead.<br>
>> * There are other ways of making sure tabstop is set correctly for<br>
>> python files, see  <a href="https://wiki." target="_blank">https://wiki.</a>.<a href="http://python.org/moin/Vim" target="_blank">python.org/moin/Vim</a><br>
>> <<a href="https://wiki.python.org/moin/Vim" target="_blank">https://wiki.python.org/moin/Vim</a>>.  I am a vIm user myself and have<br>
>> never used modelines.<br>
>> * We have vim modelines in only 828 out of 1213 python files in nova<br>
>> (68%), so if anyone is using modelines today, then it only works 68% of<br>
>> the time in nova<br>
>> * Why have the same config 828 times for one repo alone?  This violates<br>
>> the DRY principle (Don't Repeat Yourself).<br>
>><br>
>><br>
>> Related Patches:<br>
>> <a href="https://review.openstack.org/#/c/51295/" target="_blank">https://review.openstack.org/#/c/51295/</a><br>
>> <a href="https://review.openstack.org/#/q/status:open+project:openstack/nova+branch:master+topic:noboilerplate,n,z" target="_blank">https://review.openstack.org/#/q/status:open+project:openstack/nova+branch:master+topic:noboilerplate,n,z</a><br>


><br>
> I agree with everything - both not caring about this topic really, and<br>
> that we should just kill them and be done with it. Luckily, this is a<br>
> suuper easy global search and replace.<br>
><br>
> Also, since we gate on pep8, if your editor is configured incorrectly,<br>
> you'll figure it out soon enough.<br>
><br>
> _______________________________________________<br>
> OpenStack-dev mailing list<br>
> <a href="mailto:OpenStack-dev@lists.openstack.org">OpenStack-dev@lists.openstack.org</a><br>
> <a href="http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev" target="_blank">http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev</a><br>
<br>
<br>
<br>
</div></div><span class="HOEnZb"><font color="#888888">--<br>
Davanum Srinivas :: <a href="http://davanum.wordpress.com" target="_blank">http://davanum.wordpress.com</a><br>
</font></span><div class="HOEnZb"><div class="h5"><br>
_______________________________________________<br>
OpenStack-dev mailing list<br>
<a href="mailto:OpenStack-dev@lists.openstack.org">OpenStack-dev@lists.openstack.org</a><br>
<a href="http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev" target="_blank">http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev</a><br>
</div></div></blockquote></div><br><br clear="all"><div><br></div>-- <br><br><div>Kind regards, Yuriy.</div>
</div>