[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[bug#60802] [PATCH v3 1/2] platforms: Raise an exception when no suitabl
From: |
Ludovic Courtès |
Subject: |
[bug#60802] [PATCH v3 1/2] platforms: Raise an exception when no suitable platform is found. |
Date: |
Tue, 17 Jan 2023 09:59:46 +0100 |
User-agent: |
Gnus/5.13 (Gnus v5.13) Emacs/28.2 (gnu/linux) |
Hi Josselin,
Josselin Poiret <dev@jpoiret.xyz> skribis:
> This looks good to me, although in the grand scheme of things I wonder
> if that change is a step forward: for those kinds of procedures, we
> could expect consumers to instead always properly handle the #f case
> themselves, rather than baby-sitting them and systematically relying on
> exceptions in the parent procedure, no? As a caricatural example: the
> SRFI-1 `find` could raise an exception instead of returning #f, but I
> don't think anyone would consider that proper behaviour.
I share this sentiment in general (plus the fact that we should keep UI
aspects, such as error reports, separate from core logic). Here there’s
a precedent with other lookup procedures though
(‘lookup-bootloader-by-name’, ‘lookup-compressor’,
‘lookup-image-type-by-name’, etc.), so I think it’s okay to keep it that
way.
Ludo’.
[bug#60802] [PATCH v3 1/2] platforms: Raise an exception when no suitable platform is found., Maxim Cournoyer, 2023/01/14
[bug#60802] [PATCH v4 0/2] Remove unsupported u-boot-malta package, Maxim Cournoyer, 2023/01/17