Hi Maxim, On Mon, 16 Jan 2023 at 12:46, Maxim Cournoyer <[email protected]> wrote:
>> So far the we use (srfi srfi-35) exclusively to define condition types; >> I think we should do the same here, for consistency. > > Could we instead start migrating away from srfi-35 to (ice-9 > exceptions), which is the new native way to use exceptions in Guile? Well, I think this should be discussed separately. Therefore, [email protected] and +guix-devel. :-) > I > think it'd be nicer to use it in the future, to avoid newcomers being > confusing about the 3 or 4 ways to manage exceptions in Guile > (recommended read on the topic: > https://vijaymarupudi.com/blog/2022-02-13-error-handling-in-guile.html). In Guile maybe several but only one in Guix: (srfi srfi-35). :-) > Migrating the whole code at once doesn't seem a good idea, so gradually > transitioning (such as using (ice-9 exceptions) for new code) appears a > good idea to me, if we agree on the direction! Just to note that all Guix is using (srfi srfi-35); except, --8<---------------cut here---------------start------------->8--- 1 candidates: ./guix/build/minetest-build-system.scm:26: #:use-module (ice-9 exceptions) --8<---------------cut here---------------end--------------->8--- Therefore, this plan needs to be a bit cooked, otherwise it will become quickly a big mess with 2 ways for managing exceptions – and people, at least me, will be confused to pick the correct one depending on the module. Well, if such migration makes sense (I do not know?), from my point of view, the migration of the whole code in a short period of time appears to me the best; probably with a dedicated WIP branch. Cheers, simon
