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

[GNU bug Tracking System]

Your message dated Thu, 04 May 2017 22:47:00 +0300
with message-id <address@hidden>
and subject line Re: bug#26693: [PATCH 0/2] Fix indentation in indent-code.el 
and dir-locals.el
has caused the debbugs.gnu.org bug report #26693,
regarding [PATCH 0/2] Fix indentation in indent-code.el and dir-locals.el
to be marked as done.

(If you believe you have received this mail in error, please contact
address@hidden)


-- 
26693: http://debbugs.gnu.org/cgi/bugreport.cgi?bug=26693
GNU Bug Tracking System
Contact address@hidden with problems
> --- Begin Message ---
>
>
>
> Subject: 
>
> [PATCH 0/2] Fix indentation in indent-code.el and dir-locals.el
>
>
>
>
> Date: 
>
> Fri, 28 Apr 2017 16:28:53 +0300
>
>
>
>
> User-agent: 
>
> Gnus/5.13 (Gnus v5.13) Emacs/25.1 (gnu/linux)
>
>
>
>
> 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
>
>
>
> --- End Message ---
> --- Begin Message ---
>
>
>
> Subject: 
>
> Re: bug#26693: [PATCH 0/2] Fix indentation in indent-code.el and dir-locals.el
>
>
>
>
> Date: 
>
> Thu, 04 May 2017 22:47:00 +0300
>
>
>
>
> User-agent: 
>
> Gnus/5.13 (Gnus v5.13) Emacs/25.1 (gnu/linux)
>
>
>
>
> Ludovic Courtès (2017-05-02 23:39 +0200) wrote:
>
> > Hi Alex,
> >
> > Alex Kost <address@hidden> 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 <address@hidden>
> >> 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
>
>
> --- End Message ---