Alex Kost <alez...@gmail.com> skribis: > Ludovic Courtès (2015-10-02 00:34 +0300) wrote: > >> Alex Kost <alez...@gmail.com> skribis: >> >>> Ludovic Courtès (2015-09-29 22:16 +0300) wrote: >>> >>>> Alex Kost <alez...@gmail.com> skribis: >>>> >>> [...] >>>>> So I think .dir-locals.el is not the proper place for indentation rules. >>>>> 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’s 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 lack >>>> of a better solution, I’m fine having some of the rules duplicated. >>>> Specifically, rules for ‘package’, ‘origin’, ‘operating-system’, >>>> ‘substitute*’, ‘with-store’, ‘with-monad’, ‘run-with-store’, >>>> ‘run-with-state’, and ‘m…’. >>>> >>>> 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’t 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’s interesting to have them in .dir-locals.el for people who hack Guix itself. But it’s 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’.
