[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[bug#61255] bug#61841: ‘guix shell’ computes different package derivatio
From: |
Maxim Cournoyer |
Subject: |
[bug#61255] bug#61841: ‘guix shell’ computes different package derivation than ‘guix build’ |
Date: |
Mon, 27 Feb 2023 21:25:52 -0500 |
User-agent: |
Gnus/5.13 (Gnus v5.13) Emacs/28.2 (gnu/linux) |
Hi Ludo,
Ludovic Courtès <ludo@gnu.org> writes:
> Maxim Cournoyer <maxim.cournoyer@gmail.com> skribis:
>
>> Ludovic Courtès <ludo@gnu.org> writes:
>
> [...]
>
>>> Commit 68775338a510f84e63657ab09242d79e726fa457 turned out to have
>>> unintended side effects:
>>>
>>> https://issues.guix.gnu.org/61841
>>
>> Ugh.
>>
>>> I fixed it with:
>>>
>>> a516a0ba93 gexp: computed-file: Do not honor %guile-for-build.
>>> fee1d08f0d pack: Make sure tests can run without a world rebuild.
>>>
>>> Please take a look.
>>
>> Thank you. I still think it'd be nicer if computed-file had a means to
>> honor %guile-for-build rather than having to accommodate it specially as
>> you did in fee1d08f0d, so that it'd be symmetrical to gexp->derivation
>> in that regard. Why can't they?
>
> Like I wrote, ‘default-guile’ returns a package whereas
> ‘%guile-for-build’ returns a derivation.
>
> The latter is inherently lower-level: it’s used together with the
> monadic interface or with plain ‘derivation’, when we know which system
> we’re targeting. The former is higher-level, system-independent; it
> must be used for <computed-file> and similar forms, which are
> system-independent.
I see, it's starting to make sense. I'll sleep on it :-).
--
Thanks,
Maxim
- [bug#61255] [PATCH 2/5] gexp: computed-file: Honor %guile-for-build., (continued)
- [bug#61255] [PATCH 2/5] gexp: computed-file: Honor %guile-for-build., Maxim Cournoyer, 2023/02/03
- [bug#61255] [PATCH 2/5] gexp: computed-file: Honor %guile-for-build., Ludovic Courtès, 2023/02/03
- [bug#61255] [PATCH 2/5] gexp: computed-file: Honor %guile-for-build., Maxim Cournoyer, 2023/02/03
- [bug#61255] [PATCH 0/5] Add support for the RPM format to "guix pack", Ludovic Courtès, 2023/02/12
- [bug#61255] [PATCH 0/5] Add support for the RPM format to "guix pack", Maxim Cournoyer, 2023/02/16
- [bug#61255] (%guile-for-build) default in ‘computed-file’, Ludovic Courtès, 2023/02/23
- [bug#61255] (%guile-for-build) default in ‘computed-file’, Maxim Cournoyer, 2023/02/23
- [bug#61255] [PATCH 0/5] Add support for the RPM format to "guix pack", Ludovic Courtès, 2023/02/27
- [bug#61255] [PATCH 0/5] Add support for the RPM format to "guix pack", Maxim Cournoyer, 2023/02/27
- [bug#61255] bug#61841: ‘guix shell’ computes different package derivation than ‘guix build’, Ludovic Courtès, 2023/02/27
- [bug#61255] bug#61841: ‘guix shell’ computes different package derivation than ‘guix build’,
Maxim Cournoyer <=
[bug#61255] [PATCH 5/5] pack: Add RPM format., Maxim Cournoyer, 2023/02/03
[bug#61255] [PATCH 0/5] Add support for the RPM format to "guix pack", Ludovic Courtès, 2023/02/12
[bug#61255] [PATCH v2 0/8] Add support for the RPM format to "guix pack", Maxim Cournoyer, 2023/02/16