Open Stack

Pádraig Brady <P at draigBrady.com> writes:

> On 05/28/2014 08:16 AM, Martin Geisler wrote:
>> Hi everybody,
>> 
>> I'm trying to get my feet wet with OpenStack development, so I recently
>> tried to submit some small patches. One small thing I noticed was that
>> some files used
>> 
>>   # -*- encoding: utf-8 -*-
>> 
>> to specify the file encoding for both Python and Emacs. Unfortunately,
>> Emacs expects you to set "coding", not "encoding". Python is fine with
>> either. I submitted a set of patches for this:
>> 
>> * https://review.openstack.org/95862
>> * https://review.openstack.org/95864
>> * https://review.openstack.org/95865
>> * https://review.openstack.org/95869
>> * https://review.openstack.org/95871
>> * https://review.openstack.org/95880
>> * https://review.openstack.org/95882
>> * https://review.openstack.org/95886
>> 
>> It was pointed out to me that such a change ought to be coordinated
>> better via bug(s) or the mailinglist, so here I am :)
>
> This is valid change.
> I don't see why there is any question
> as it only improves the situation for emacs
> which will pop up an error when trying to edit these files.

Yes, exactly :)

It's also worth noting that the files *already* use Emacs-specific
markup in the form of the "-*-" markers. Python doesn't care about those
at all, but Emacs relies on them.

The reviewers in https://review.openstack.org/95886/ suggests removing
the coding lines completely and also remove non-ASCII characters as
necessary to make that possible.

I can definitely do that instead if people like that better. The only
problem I see is that we'll have to hope that © is the only non-ASCII
character -- I haven't checked if that's the case yet, but it's not
uncommon to find non-ASCII characters in author names as well.

> You could create a bug I suppose to reference all the changes though I
> don't think that's mandatory since this isn't user facing.

Okay, I'm not really familiar with the process around here. I hoped I
could just make some commits and submit them for review :)

-- 
Martin Geisler

https://plus.google.com/+MartinGeisler/
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 818 bytes
Desc: not available
URL: <http://lists.openstack.org/pipermail/openstack-dev/attachments/20140528/4b8b9354/attachment.pgp>