Re: [PATCH libosinfo 2/2] Remove emacs 'Local variables' annotations

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On 3/4/19 5:08 AM, Daniel P. Berrangé wrote:
> On Sat, Mar 02, 2019 at 06:35:18PM -0500, Cole Robinson wrote:
>> No other virt projects have annotations like this, plus IMO
>> .editorconfig handles this in a much nicer way and more flexible
>> way. Ditch them
> 
> .editorconfig requires users to install extra packages to work.
> 
> If we want to remove emacs annotations, then we need to have
> a .dir-locals.el file defined, so that correct indentation is
> used out of the box with emacs.
> 
> 
> Regards,
> Daniel
> 

Does the attached patch work for you? I've never used emacs so I don't
trust my testing

- Cole
>From 1a716cbbe5a32f829cf90ead6dfdceefd8d620f3 Mon Sep 17 00:00:00 2001
Message-Id: <1a716cbbe5a32f829cf90ead6dfdceefd8d620f3.1551904198.git.crobinso@xxxxxxxxxx>
From: Cole Robinson <crobinso@xxxxxxxxxx>
Date: Wed, 6 Mar 2019 15:27:42 -0500
Subject: [PATCH libosinfo] Add toplevel .dir-locals.el

This centralizes emacs config in a top level file, matching the
bits sprinkled around in 'Local variable' comments

Signed-off-by: Cole Robinson <crobinso@xxxxxxxxxx>
---
 .dir-locals.el | 7 +++++++
 1 file changed, 7 insertions(+)
 create mode 100644 .dir-locals.el

diff --git a/.dir-locals.el b/.dir-locals.el
new file mode 100644
index 0000000..85795b3
--- /dev/null
+++ b/.dir-locals.el
@@ -0,0 +1,7 @@
+(
+ (c-mode . (
+            (indent-tabs-mode . nil)
+            (c-indent-level . 4)
+            (c-basic-offset . 4)
+            ))
+ )
-- 
2.20.1

_______________________________________________
Libosinfo mailing list
Libosinfo@xxxxxxxxxx
https://www.redhat.com/mailman/listinfo/libosinfo

[Index of Archives]     [Virt Tools]     [Libvirt Users]     [Fedora Users]     [Fedora Maintainers]     [Fedora Desktop]     [Fedora SELinux]     [Big List of Linux Books]     [Yosemite News]     [KDE Users]

  Powered by Linux