Ludovic Courtès <[email protected]> writes:

> Hi Tomas,
>
> Tomas Volf <[email protected]> skribis:
>
>> --8<---------------cut here---------------start------------->8---
>> ice-9/eval.scm:223:20: In procedure proc:
>> error: guix-channel?: unbound variable
>> hint: Did you forget `(use-modules (guix channels))' or `#:use-module (guix 
>> channels)'?
>> --8<---------------cut here---------------end--------------->8---
>
> The suggestion is bogus in this case: ‘use-modules’ is not allowed.
>
>> If I understand it right, that is because guix-channel? is not part of
>> the new sandbox.  However I am not sure hot to solve this.
>
> It’s a mistake, we should add ‘guix-channel?’ to the set of default
> bindings.  Here’s the proposed fix:
>
>   https://codeberg.org/guix/guix/pulls/8562

Thank you for the prompt fix!  I can confirm it works with my channel
file, so:

Reviewed-by: Tomas Volf <[email protected]>

I am getting

--8<---------------cut here---------------start------------->8---
channels-base.sex:1:7: warning: channel 'guix' is not trusted
channels-base.sex:9:7: warning: channel 'nonguix' is not trusted
channels-base.sex:17:7: warning: channel 'wolfsden' is not trusted
--8<---------------cut here---------------end--------------->8---

but, per documentation, that is now expected.  I am not sure I follow
why the warning is necessary even for local files, but it is only
warning, so I can simply ignore it I guess.

>
> In the meantime, there are two ways to work around it:
>
>   • (filter (lambda (channel) (not (eq? (channel-name channel) ’guix))) …)
>
>   • or use ‘--unsafe-channel-evaluation’ until the fix above is merged.

I will just apply your patch to my fork :)

Tomas

-- 
There are only two hard things in Computer Science:
cache invalidation, naming things and off-by-one errors.

Reply via email to