Subject: Re: bug#61977: [PATCH] channels: Raise an error if dependency's introduction is malformed

From: Josselin Poiret <[email-masked]>
Date: Sat, 18 Mar 2023 22:42:01 +0100

Hi Ludo,

Ludovic Courtès <[email-masked]> writes:

> Hi Josselin,
>
> Josselin Poiret <[email-masked]> skribis:
>
>> * guix/channels.scm (sexp->channel-introduction): Do it.
>
> To be clear, without this a malformed introduction sexp would be
> silently ignored; consequently, the channel in question would not be
> authenticated, in which case the “channel '~a' lacks 'introduction'
> field” warning would be printed.  Am I right?
>
> So the goal here is to catch errors in ‘.guix-channel’ earlier, right?

Exactly!  Someone else reported this elsewhere because they were using
the syntax for a channels.scm file instead of the channel dependencies
syntax, which is different, and consumers of that channel kept getting
warning messages.

Best,
-- 
Josselin Poiret