From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([2001:4830:134:3::10]:41118) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1d5fWf-0006WL-J1 for guix-patches@gnu.org; Tue, 02 May 2017 17:40:06 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1d5fWc-0006Ep-BY for guix-patches@gnu.org; Tue, 02 May 2017 17:40:05 -0400 Received: from debbugs.gnu.org ([208.118.235.43]:53445) by eggs.gnu.org with esmtps (TLS1.0:RSA_AES_128_CBC_SHA1:16) (Exim 4.71) (envelope-from ) id 1d5fWc-0006El-81 for guix-patches@gnu.org; Tue, 02 May 2017 17:40:02 -0400 Received: from Debian-debbugs by debbugs.gnu.org with local (Exim 4.84_2) (envelope-from ) id 1d5fWc-0005MC-36 for guix-patches@gnu.org; Tue, 02 May 2017 17:40:02 -0400 Subject: bug#26693: [PATCH 0/2] Fix indentation in indent-code.el and dir-locals.el Resent-Message-ID: From: ludo@gnu.org (Ludovic =?UTF-8?Q?Court=C3=A8s?=) References: <878tmkacfe.fsf@gmail.com> Date: Tue, 02 May 2017 23:39:24 +0200 In-Reply-To: <878tmkacfe.fsf@gmail.com> (Alex Kost's message of "Fri, 28 Apr 2017 16:28:53 +0300") Message-ID: <87k25zym43.fsf@gnu.org> MIME-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: quoted-printable List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: guix-patches-bounces+kyle=kyleam.com@gnu.org Sender: "Guix-patches" To: Alex Kost Cc: 26693@debbugs.gnu.org Hi Alex, Alex Kost skribis: > As reported by Petter on #guix, "indent-code.el" may not work, and it is > absolutely not clear why it happens (it took me 20 minutes to figure out > what the problem was). > > So the problem is that "indent-code.el" works only when it is run from > the top directory of the guix checkout (as it is described in the > manual), otherwise it doesn't indent properly (because it doesn't find > ".dir-locals.el"). > > So the upcoming patch fixes this problem. Also there is another patch > that indents 'modify-keywords' properly. Good catch! > * .dir-locals.el: Add indentation rules for 'replace', 'add-before' and > 'add-after'. OK. > >From d4abac119e2c62cc42a8786034c905196d7262e6 Mon Sep 17 00:00:00 2001 > From: Alex Kost > Date: Fri, 28 Apr 2017 16:07:33 +0300 > Subject: [PATCH] etc: indent-code.el: Define source tree by "indent-code.= el". > > * etc/indent-code.el.in: Set default directory by "indent-code.el" > itself, otherwise it works only when called from the top directory of > the guix source tree. LGTM. Thank you! Ludo=E2=80=99.