GNU bug report logs - #53505
[PATCH] .dir-locals.el: Don't mess up indentation of prepend and append.

Previous Next

Package: guix-patches;

Reported by: Liliana Marie Prikler <liliana.prikler <at> gmail.com>

Date: Mon, 24 Jan 2022 16:56:01 UTC

Severity: normal

Tags: patch

Done: Liliana Marie Prikler <liliana.prikler <at> gmail.com>

Bug is archived. No further changes may be made.

Full log


Message #11 received at 53505 <at> debbugs.gnu.org (full text, mbox):

From: Liliana Marie Prikler <liliana.prikler <at> gmail.com>
To: Ludovic Courtès <ludo <at> gnu.org>
Cc: 53505 <at> debbugs.gnu.org
Subject: Re: bug#53505: [PATCH] .dir-locals.el: Don't mess up indentation of
 prepend and append.
Date: Wed, 26 Jan 2022 20:26:21 +0100
Hi!

Am Mittwoch, dem 26.01.2022 um 12:24 +0100 schrieb Ludovic Courtès:
> Hi!
> 
> Liliana Marie Prikler <liliana.prikler <at> gmail.com> skribis:
> 
> > The current version appears to assume special values in the first
> > and second package, but it's either all lists (default append) or
> > all package input expressions (inside modify-inputs), which
> > themselves are either packages or a list consisting of a package
> > and a string.
> > 
> > * .dir-locals.el: Drop (eval . (put 'prepend 'scheme-indent-
> > function 2))
> > and (eval . (put 'append 'scheme-indent-function 2)).
> 
> Yeah it’s been bothering me too.  :-)
> 
> We can keep ‘prepend’ though as it doesn’t clash with anything.
It might not clash, but is its behaviour correct?  IIRC, prepend and
append take a list of packages inside modify-inputs, so if I wanted to
add spam, ham, eggs, milk and butter to a python-package, I'd write
that as

(modify-inputs my-package
  (prepend python-spam
       python-ham
     python-eggs
     python-milk
     python-butter))

That doesn't seem right, now does it?




This bug report was last modified 3 years and 113 days ago.

Previous Next


GNU bug tracking system
Copyright (C) 1999 Darren O. Benham, 1997,2003 nCipher Corporation Ltd, 1994-97 Ian Jackson.