GNU bug report logs - #61013
guix style indentation problem in (list ...)

Previous Next

To add a comment to this bug, you must first unarchive it, by sending
a message to control AT debbugs.gnu.org, with unarchive 61013 in the body.
You can then email your comments to 61013 AT debbugs.gnu.org in the normal way.

Toggle the display of automated, internal messages from the tracker.

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

From: Maxim Cournoyer <maxim.cournoyer <at> gmail.com>
To: bug-guix <bug-guix <at> gnu.org>
Subject: guix style indentation problem in (list ...)
Date: Sun, 22 Jan 2023 19:38:28 -0500

Hello,

I recently stumbled on this indentation produced with guix style:

--8<---------------cut here---------------start------------->8---
    (arguments
     (list
           ;; Disable tests that use the network and non-free assets.
           #:test-flags #~(list "-m" "not external and not samples")))
--8<---------------cut here---------------end--------------->8---

While the conventional way would be to indent it as:

--8<---------------cut here---------------start------------->8---
    (arguments
     (list
      ;; Disable tests that use the network and non-free assets.
      #:test-flags #~(list "-m" "not external and not samples")))
--8<---------------cut here---------------end--------------->8---

-- 
Thanks,
Maxim

Message #10 received at 61013-done <at> debbugs.gnu.org (full text, mbox):

From: Ludovic Courtès <ludo <at> gnu.org>
To: Maxim Cournoyer <maxim.cournoyer <at> gmail.com>
Cc: 61013-done <at> debbugs.gnu.org
Subject: Re: bug#61013: guix style indentation problem in (list ...)
Date: Tue, 07 Mar 2023 14:59:09 +0100

Hello,

Maxim Cournoyer <maxim.cournoyer <at> gmail.com> skribis:

> I recently stumbled on this indentation produced with guix style:
>
>     (arguments
>      (list
>            ;; Disable tests that use the network and non-free assets.
>            #:test-flags #~(list "-m" "not external and not samples")))
>
>
> While the conventional way would be to indent it as:
>
>     (arguments
>      (list
>       ;; Disable tests that use the network and non-free assets.
>       #:test-flags #~(list "-m" "not external and not samples")))

Fixed in 0a37921d851b94aef6900214098dc5bd62b46e26, thanks!

Ludo’.

Message #13 received at 61013-done <at> debbugs.gnu.org (full text, mbox):

From: Maxim Cournoyer <maxim.cournoyer <at> gmail.com>
To: Ludovic Courtès <ludo <at> gnu.org>
Cc: 61013-done <at> debbugs.gnu.org
Subject: Re: bug#61013: guix style indentation problem in (list ...)
Date: Tue, 07 Mar 2023 13:31:29 -0500

Hi Ludo,

Ludovic Courtès <ludo <at> gnu.org> writes:

> Hello,
>
> Maxim Cournoyer <maxim.cournoyer <at> gmail.com> skribis:
>
>> I recently stumbled on this indentation produced with guix style:
>>
>>     (arguments
>>      (list
>>            ;; Disable tests that use the network and non-free assets.
>>            #:test-flags #~(list "-m" "not external and not samples")))
>>
>>
>> While the conventional way would be to indent it as:
>>
>>     (arguments
>>      (list
>>       ;; Disable tests that use the network and non-free assets.
>>       #:test-flags #~(list "-m" "not external and not samples")))
>
> Fixed in 0a37921d851b94aef6900214098dc5bd62b46e26, thanks!

Awesome, that was quick!  I'll try to use it more regularly and reports
of any problematic behavior.

-- 
Thanks,
Maxim

Previous Next

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