[Top][All Lists]

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

Re: [PATCH 2/2] gnu: Add openvpn service.

From: Alex Kost
Subject: Re: [PATCH 2/2] gnu: Add openvpn service.
Date: Thu, 19 Jan 2017 22:01:46 +0300
User-agent: Gnus/5.13 (Gnus v5.13) Emacs/25.1 (gnu/linux)

Ludovic Courtès (2017-01-19 12:51 +0100) wrote:

> Alex Kost <address@hidden> skribis:
>> Ludovic Courtès (2017-01-15 23:11 +0100) wrote:
>>> I see.  So maybe we should just add a rule for ‘match’ in
>>> .dir-locals.el; any downside to that?
>> I'm against it, as I think ".dir-locals.el" should contain only elisp
>> stuff *specific* to a project.  So I would rather do an opposite, i.e. I
>> would remove 'lambda*', 'eval-when', 'call-with-prompt',
>> 'call-with-input-string' and 'test-*' from the .dir-locals (all of these
>> keywords are handled by Geiser but not by scheme-mode).
> Yeah.
>> But if you consider to add it, then also add 'when', 'unless',
>> 'match-lambda', 'case-lambda', 'and-let*', 'with-output-to-string',
>> 'with-input-from-string' 'call-with-output-string'.  All these are not
>> handled by scheme-mode (but handled by Geiser).
> I don’t know, do you think we should put these in a different file?  How
> would that work?

As I said, my opinion is that ".dir-locals.el" shouldn't contain any
keyword that is not specific to Guix, so...

> The problem is that people using indent-code.el typically have raw Emacs
> with scheme-mode, but most likely without Geiser and Emacs-Guix.  So we
> need a mechanism that works in that case.

... I see 2 ways:

- either using "guix environment emacs-guix" as Ricardo proposed

- or simply by adding the required indentation rules for all the missing
  keywords directly into "indent-code.el".

I prefer the latter as it seems simple and clean.


reply via email to

[Prev in Thread] Current Thread [Next in Thread]