Subject: [bug#62153] [PATCH 4/5] guix: pack: Build layered images.

From: Mathieu Othacehe <[email-masked]>
Date: Wed, 27 Dec 2023 21:25:02 +0100

> +If MAX-LAYERS is not false, the image will with many of the store paths being
> +on their own layer to improve sharing between images."

Same comment as in previous patch, "many" is a bit vague. "the image
will be splitted in up to MAX-LAYERS layers" or so could work.

> +         (check   (gexp->derivation "check-tarball"
> +                                    (with-imported-modules '((guix build utils))

You could rearrange as:

--8<---------------cut here---------------start------------->8---
(check   (gexp->derivation
          "check-tarball"
          (with-imported-modules '((guix build utils))
--8<---------------cut here---------------end--------------->8---

to reduce the indentation of this block:

> +                                      #~(begin
> +                                          (use-modules (guix build utils)
> +                                                       (ice-9 match))

...

> +                                                   (string=? (string-append #$%bootstrap-guile "/bin")
> +                                                             (pk 'binlink (readlink bin)))
> +                                                   (string=? (string-append #$profile "/bin/guile")
> +                                                             (pk 'guilelink (readlink "bin/Guile"))))

Left over pk's?

Thanks,

Mathieu