[bug#61255] [PATCH 0/5] Add support for the RPM format to "guix pack"

[Ludovic Courtès]

Hi,

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

> Ludovic Courtès <ludo@gnu.org> writes:
>
>> Hi,
>>
>> Maxim Cournoyer <maxim.cournoyer@gmail.com> skribis:
>>
>>> * tests/pack.scm: Fix indentation.
>>
>> [...]
>>
>>>         (check   (gexp->derivation
>>> -                 "check-tarball"
>>> -                 (with-imported-modules '((guix build utils))
>>
>> [...]
>>
>>> +                    "check-tarball"
>>> +                  (with-imported-modules '((guix build utils))
>>
>> I’m not convinced by the indentation rule for ‘gexp->derivation’ added
>> in 82daab42811a2e3c7684ebdf12af75ff0fa67b99: there’s no reason to treat
>> ‘gexp->derivation’ differently from other procedures.
>
> The benefit I saw was that writing
>
>  (gexp->derivation the-name
>   #~(begin
>      (the
>        (multi-line
>         (gexp)))))

I understand, but you know, it’s best to avoid unilaterally changing
established conventions.  :-)

If and when there’s consensus about this change, (guix read-print)
should be updated.

Ludo’.