From mboxrd@z Thu Jan 1 00:00:00 1970 From: ludo@gnu.org (Ludovic =?utf-8?Q?Court=C3=A8s?=) Subject: Re: [PATCH 2/2] gnu: Add openvpn service. Date: Thu, 19 Jan 2017 12:51:33 +0100 Message-ID: <87mvenz2re.fsf@gnu.org> References: <20170112175714.3855-1-julien@lepiller.eu> <20170112175714.3855-2-julien@lepiller.eu> <87bmvarged.fsf@gnu.org> <20170114143938.45d8f59e@lepiller.eu> <87pojpjq9m.fsf@gnu.org> <878tqcrxih.fsf@gmail.com> <87wpdwdld6.fsf@gnu.org> <87bmv6243a.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]:39527) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1cUBFk-0003xY-NO for guix-devel@gnu.org; Thu, 19 Jan 2017 06:51:41 -0500 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1cUBFf-0000Fw-RV for guix-devel@gnu.org; Thu, 19 Jan 2017 06:51:40 -0500 In-Reply-To: <87bmv6243a.fsf@gmail.com> (Alex Kost's message of "Mon, 16 Jan 2017 22:32:25 +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" To: Alex Kost Cc: guix-devel@gnu.org Alex Kost skribis: > Ludovic Court=C3=A8s (2017-01-15 23:11 +0100) wrote: > >> Alex Kost skribis: >> >>> Ludovic Court=C3=A8s (2017-01-14 22:16 +0100) wrote: >>> >>>>> +(define (uglify-field-name name) >>>>> + (match name >>>>> + ('verbosity "verb") >>>>> + (_ (let ((str (symbol->string name))) >>>>> + (if (string-suffix? "?" str) >>>>> + (substring str 0 (1- (string-length str))) >>>>> + str))))) >>>> >>>> Not sure why but indent-code.el got =E2=80=98match=E2=80=99 indentatio= n wrong, although >>>> it works for me. Alex, any ideas? >>> >>> Because 'scheme-mode' does not provide indentation rules for 'match' >>> (and many other keywords). Geiser indents 'match' properly for you. >> >> I see. So maybe we should just add a rule for =E2=80=98match=E2=80=99 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=E2=80=99t know, do you think we should put these in a different file?= How would that work? 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. Thanks for your feedback! Ludo=E2=80=99.