guix-patches
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

[bug#57680] [PATCH 0/2] image: Add tarball support.


From: Ludovic Courtès
Subject: [bug#57680] [PATCH 0/2] image: Add tarball support.
Date: Sat, 24 Sep 2022 15:50:22 +0200
User-agent: Gnus/5.13 (Gnus v5.13) Emacs/28.1 (gnu/linux)

Mathieu Othacehe <othacehe@gnu.org> skribis:

> Move the compression record to a dedicated module so that it can be used
> outside (guix scripts pack) module.
>
> * guix/scripts/pack.scm (<compressor>, %compressors, lookup-compressor): Move
> it to ...
> * guix/compression.scm: ... this new file.
> * Makefile.am (MODULES): Add it.

I’m pretty sure I commented on this patch as part of another series
recently but I can’t find it anymore.

The guts of it is:

  1. (guix compression) sounds like it could just as well be about
     offering an abstraction over guile-{zlib,zstd,lzlib} like that
     currently in (guix utils).  So the name is misleading.

  2. We cannot refer to (gnu …) from (guix …) or, if we really need to
     do so, then that should happen lazily at run time (do not miss
     Josselin’s excellent guided tour at the Ten Years, which included a
     discussion of this! :-)).

Hope that makes sense!

Ludo’.





reply via email to

[Prev in Thread] Current Thread [Next in Thread]