Markus Armbruster <arm...@redhat.com> writes:
> Alex Bennée <alex.ben...@linaro.org> writes:
>
>> Victor Toso <victort...@redhat.com> writes:
>>
>>> Similar to f7160f3218 "schemas: Add vim modeline"
>>>
>>> Signed-off-by: Victor Toso <victort...@redhat.com>
>>> ---
>>> qapi/audio.json | 1 +
>>> qapi/compat.json | 1 +
>>> qapi/replay.json | 1 +
>>> qapi/trace.json | 1 +
>>> 4 files changed, 4 insertions(+)
>>>
>>> diff --git a/qapi/audio.json b/qapi/audio.json
>>> index 9cba0df8a4..fe3b506ec3 100644
>>> --- a/qapi/audio.json
>>> +++ b/qapi/audio.json
>>> @@ -1,4 +1,5 @@
>>> # -*- mode: python -*-
>>> +# vim: filetype=python
>>
>> I realise we have Emacs mode annotations here as well and I'm not going
>> to start a holy war but I thought we used .editorconfig to avoid messing
>> directly with the individual files. So something like:
>>
>> [*.json]
>> indent_style = space
>> emacs_mode = python
>> vim_filetype = python
>>
>> should also do the job.
>
> Most schema files have lines for both Emacs and vim. Some have only the
> former, and this patch adds the latter to them. Seems fair to me.
No argument from me there.
> Perhaps we could get rid of them all instead. As far as I can tell,
> .editorconfig doesn't work for stock Emacs out of the box, though.
It does require a package-install although fortunately it's in non-gnu
elpa so should work out of the box with any reasonably recent Emacs. I
personally wrap this stuff up in use-package:
;; EditorConfig
(use-package editorconfig
:ensure t
:diminish "EdCf"
:config
;; See https://github.com/editorconfig/editorconfig-emacs/issues/246
(add-to-list 'editorconfig-exclude-modes 'git-rebase-mode)
(editorconfig-mode 1))
We also have have a rather embryonic .dir-locals in the root of the
source tree as well.
Our vim-brethren seem to have multiple ways of installing their plugin:
https://github.com/editorconfig/editorconfig-vim#installation
--
Alex Bennée
