From mboxrd@z Thu Jan 1 00:00:00 1970 From: Alex Kost Subject: Re: .dir-locals.el vs. guix-devel-mode Date: Fri, 02 Oct 2015 15:46:56 +0300 Message-ID: <87h9m9xwsf.fsf@gmail.com> References: <871tdm6i0i.fsf@gmail.com> <8737y28fss.fsf@gnu.org> <87d1x57yhg.fsf@gmail.com> <87si5zwqqu.fsf@gnu.org> <878u7qybjn.fsf_-_@gmail.com> <87r3lhxchs.fsf@gnu.org> <87d1wyqoqw.fsf@gmail.com> <87wpv6l1d8.fsf@gnu.org> Mime-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: quoted-printable Return-path: Received: from eggs.gnu.org ([2001:4830:134:3::10]:60252) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1Zhzjm-0006MZ-I8 for guix-devel@gnu.org; Fri, 02 Oct 2015 08:46:59 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1Zhzjj-0001MX-Aa for guix-devel@gnu.org; Fri, 02 Oct 2015 08:46:58 -0400 In-Reply-To: <87wpv6l1d8.fsf@gnu.org> ("Ludovic \=\?utf-8\?Q\?Court\=C3\=A8s\=22'\?\= \=\?utf-8\?Q\?s\?\= message of "Thu, 01 Oct 2015 23:34:11 +0200") List-Id: "Development of GNU Guix and the GNU System distribution." List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: guix-devel-bounces+gcggd-guix-devel=m.gmane.org@gnu.org Sender: guix-devel-bounces+gcggd-guix-devel=m.gmane.org@gnu.org To: Ludovic =?utf-8?Q?Court=C3=A8s?= Cc: guix-devel@gnu.org Ludovic Court=C3=A8s (2015-10-02 00:34 +0300) wrote: > Alex Kost skribis: > >> Ludovic Court=C3=A8s (2015-09-29 22:16 +0300) wrote: >> >>> Alex Kost skribis: >>> >> [...] >>>> So I think .dir-locals.el is not the proper place for indentation rule= s. >>>> IMHO they should be moved to "guix-devel.el" and the manual should >>>> recommend using 'guix-devel-mode' for editing guix package files as it >>>> provides the proper indenting, highlighting and some useful key >>>> bindings. >>> >>> Yes, but I agree with Taylan: a passerby should get a reasonable setup >>> in place automatically. That=E2=80=99s what I like about .dir-locals.e= l: it >>> allows you to make sure that a minimum set of rules is in place, which >>> in turn means that patches are more likely to come out right, which >>> means less frustration and increased happiness. >>> >>> Using eval + load in .dir-locals.el is indeed ugly though. So, for lack >>> of a better solution, I=E2=80=99m fine having some of the rules duplica= ted. >>> Specifically, rules for =E2=80=98package=E2=80=99, =E2=80=98origin=E2= =80=99, =E2=80=98operating-system=E2=80=99, >>> =E2=80=98substitute*=E2=80=99, =E2=80=98with-store=E2=80=99, =E2=80=98w= ith-monad=E2=80=99, =E2=80=98run-with-store=E2=80=99, >>> =E2=80=98run-with-state=E2=80=99, and =E2=80=98m=E2=80=A6=E2=80=99. >>> >>> How does that sound? >> >> I'd like to have them in "guix-devel.el", but I don't see the reason for >> duplicating. I think they either should be placed in "guix-devel.el" >> (surely the right thing for me :-)) or stay in ".dir-locals.el" (I'm >> afraid the right thing for the most :-(). >> >> Why do you suggest the duplication? > > Just to make sure that anyone using Emacs will get things right, even if > they haven=E2=80=99t learned about guix-devel.el yet. > > WDYT? I thought that most people (except me) agreed that indentation rules should stay in ".dir-locals.el", so I don't see a reason why they also need to be duplicated in "guix-devel.el". --=20 Alex