Subject: bug#26693: closed (Re: bug#26693: [PATCH 0/2] Fix indentation in indent-code.el and dir-locals.el)

From: [email-masked] (GNU bug Tracking System)
Date: Thu, 04 May 2017 19:48:02 +0000

Your bug report

#26693: [PATCH 0/2] Fix indentation in indent-code.el and dir-locals.el

which was filed against the guix-patches package, has been closed.

The explanation is attached below, along with your original report.
If you require more details, please reply to [email-masked].

-- 
26693: http://debbugs.gnu.org/cgi/bugreport.cgi?bug=26693
GNU Bug Tracking System
Contact [email-masked] with problems

Subject: Re: bug#26693: [PATCH 0/2] Fix indentation in indent-code.el and dir-locals.el
From: Alex Kost <[email-masked]>
To: [email-masked] (Ludovic Courtès)
Cc: [email-masked]
Date: Thu, 04 May 2017 22:47:00 +0300
List-Id: <debbugs-submit.debbugs.gnu.org>

Ludovic Courtès (2017-05-02 23:39 +0200) wrote:

> Hi Alex,
>
> Alex Kost <[email-masked]> 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 <[email-masked]>
>> 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!

Committed, thanks!

-- 
Alex

Subject: [PATCH 0/2] Fix indentation in indent-code.el and dir-locals.el
From: Alex Kost <[email-masked]>
To: [email-masked]
Date: Fri, 28 Apr 2017 16:28:53 +0300
List-Id: <debbugs-submit.debbugs.gnu.org>

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.

-- 
Alex