From mboxrd@z Thu Jan 1 00:00:00 1970 From: taylanbayirli@gmail.com (Taylan Ulrich =?utf-8?Q?Bay=C4=B1rl=C4=B1?= =?utf-8?Q?=2FKammer?=) Subject: Re: .dir-locals.el vs. guix-devel-mode Date: Tue, 29 Sep 2015 13:22:18 +0200 Message-ID: <871tdh1ndh.fsf@T420.taylan> References: <871tdm6i0i.fsf@gmail.com> <8737y28fss.fsf@gnu.org> <87d1x57yhg.fsf@gmail.com> <87si5zwqqu.fsf@gnu.org> <878u7qybjn.fsf_-_@gmail.com> <87eghi1f9c.fsf@T420.taylan> <87d1x1jxhi.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]:55426) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1Zgsyz-0005fF-Ni for guix-devel@gnu.org; Tue, 29 Sep 2015 07:22:06 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1Zgsyy-0003jK-Vs for guix-devel@gnu.org; Tue, 29 Sep 2015 07:22:05 -0400 In-Reply-To: <87d1x1jxhi.fsf@gmail.com> (Alex Kost's message of "Tue, 29 Sep 2015 14:06:33 +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 writes: > Taylan Ulrich "Bay=C4=B1rl=C4=B1/Kammer" (2015-09-28 23:05 +0300) wrote: > >> Alex Kost writes: >> >>> [...] >>> >>> More generally, I'm strongly against using "eval" in .dir-locals. IMO >>> this file should be used only to set local variables, and ideally it >>> shouldn't evaluate an arbitrary code. >> >> Just to clarify: I suppose you mean using eval for things other than >> which Emacs can automatically determine to be safe. E.g. >> >> (eval . (put 'foo 'scheme-indent-function )) >> >> for integer is safe; Emacs will eval that silently if I'm not >> mistaken. > > Actually I don't like .dir-locals.el in general. I consider it a > workaround for the case when there are no appropriate major/minor modes > for editing the project files. I agree a full guix-devel minor mode might be a good idea, but a passerby contributor or newcomer shouldn't need to know about it IMO. It will seem like an additional hurdle to them. If the newcomer sticks around, they'll find out and use the mode eventually, but first impressions matter a lot. For that reason I'd use .dir-locals.el to its limits, i.e. so long as the user isn't bothered with the unsafe warning. (So that would actually mean using it less than how much we use it right now...) Taylan