From mboxrd@z Thu Jan 1 00:00:00 1970 From: ludo@gnu.org (Ludovic =?utf-8?Q?Court=C3=A8s?=) Subject: Re: .dir-locals.el vs. guix-devel-mode Date: Fri, 02 Oct 2015 14:51:35 +0200 Message-ID: <87a8s1e8mg.fsf@gnu.org> 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> <87h9m9xwsf.fsf@gmail.com> 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]:33040) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1ZhzoL-0006zp-Pk for guix-devel@gnu.org; Fri, 02 Oct 2015 08:51:42 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1ZhzoI-0003SY-HU for guix-devel@gnu.org; Fri, 02 Oct 2015 08:51:41 -0400 In-Reply-To: <87h9m9xwsf.fsf@gmail.com> (Alex Kost's message of "Fri, 02 Oct 2015 15:46:56 +0300") 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: Alex Kost Cc: guix-devel@gnu.org Alex Kost skribis: > 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 rul= es. >>>>> 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.= el: 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 la= ck >>>> of a better solution, I=E2=80=99m fine having some of the rules duplic= ated. >>>> 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=98= with-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". It=E2=80=99s interesting to have them in .dir-locals.el for people who hack= Guix itself. But it=E2=80=99s also interesting to have them in guix-devel.el for people = who maintain, say, their own package collection outside of the Guix repo. Does that make sense? Ludo=E2=80=99.