[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[bug#71111] [PATCH 0/1] services: home: Use pairs instead of lists.
From: |
Maxim Cournoyer |
Subject: |
[bug#71111] [PATCH 0/1] services: home: Use pairs instead of lists. |
Date: |
Thu, 23 May 2024 12:02:42 -0400 |
User-agent: |
Gnus/5.13 (Gnus v5.13) |
Hi,
Andrew Tropin <andrew@trop.in> writes:
> On 2024-05-23 11:38, Zheng Junjie wrote:
>
>> Andrew Tropin via Guix-patches via <guix-patches@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@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 agree.
--
Thanks,
Maxim
- [bug#71111] [PATCH 0/1] services: home: Use pairs instead of lists., Andrew Tropin, 2024/05/22
- [bug#71112] [PATCH 1/1] services: home: Use pairs instead of lists., Andrew Tropin, 2024/05/22
- [bug#71111] [PATCH 1/1] services: home: Use pairs instead of lists., Andrew Tropin, 2024/05/22
- [bug#71111] [PATCH 0/1] services: home: Use pairs instead of lists., Richard Sent, 2024/05/22
- [bug#71111] [PATCH 0/1] services: home: Use pairs instead of lists., Zheng Junjie, 2024/05/22
- [bug#71111] [PATCH v2 0/1] services: home: Use pairs instead of lists., Andrew Tropin, 2024/05/23
- [bug#71111] [PATCH v2 1/1] services: home: Use pairs instead of lists., Andrew Tropin, 2024/05/23