GNU bug report logs -
#71111
[PATCH 0/1] services: home: Use pairs instead of lists.
Previous Next
Reported by: Andrew Tropin <andrew <at> trop.in>
Date: Wed, 22 May 2024 10:13:01 UTC
Severity: normal
Tags: patch
Done: Andrew Tropin <andrew <at> trop.in>
Bug is archived. No further changes may be made.
Full log
View this message in rfc822 format
[Message part 1 (text/plain, inline)]
On 2024-05-23 11:38, Zheng Junjie wrote:
> Andrew Tropin via Guix-patches via <guix-patches <at> gnu.org> writes:
>
>> After rewriting from car/cdr to match-lambda in v2 of this patch:
>> https://yhetil.org/guix-patches/3394b0b51f6a5a608ebcfb7a63fdc34e52fe928e.1711046203.git.richard <at> freakingpenguin.com/
>>
>> the format changed from pairs to lists, I didn't noticed this nuance
>> during review because the documentation still says that service should
>> be configured and extended with pairs. Also, pairs are more
>> apropriate data type here. And this match-lambda rewrite will break
>> downstream RDE user's setups after migrating to upstreamed version of
>> service.
>>
>> That's why I propose to go back to pairs.
>
> Maybe we can support pairs and list of length two at same time?
Thank you for the idea, however I think ambiguity is a bad practice,
from my early experience with guix it's more confusing rather than
helpful. I still don't know why profile-service-type accepts list of
lists rather than alist (list of pairs).
>
>>
>> Andrew Tropin (1):
>> services: home: Use pairs instead of lists.
>>
>> doc/guix.texi | 4 ++--
>> gnu/services/guix.scm | 2 +-
>> gnu/tests/guix.scm | 2 +-
>> 3 files changed, 4 insertions(+), 4 deletions(-)
>>
>>
>> base-commit: b06a460bf5236a9d52f936f2023451051b3e622a
--
Best regards,
Andrew Tropin
[signature.asc (application/pgp-signature, inline)]
This bug report was last modified 357 days ago.
Previous Next
GNU bug tracking system
Copyright (C) 1999 Darren O. Benham,
1997,2003 nCipher Corporation Ltd,
1994-97 Ian Jackson.