guix-patches
[Top][All Lists]
Advanced

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

[bug#30013] [PATCH] gnu: Add vboot-reference.


From: Ludovic Courtès
Subject: [bug#30013] [PATCH] gnu: Add vboot-reference.
Date: Mon, 08 Jan 2018 09:51:09 +0100
User-agent: Gnus/5.13 (Gnus v5.13) Emacs/25.3 (gnu/linux)

Hi!

Marius Bakke <address@hidden> skribis:

> * gnu/packages/bootloaders.scm (vboot-reference): New public variable.

“vboot-utils”

> +       #:phases (modify-phases %standard-phases
> +                  (add-after 'unpack 'patch-hard-coded-paths
> +                    (lambda* (#:key outputs #:allow-other-keys)
> +                      (substitute* "futility/misc.c"
> +                        (("/bin/cp") (which "cp")))
> +                      (substitute* "tests/bitmaps/TestBmpBlock.py"
> +                        (("/usr/bin/cmp") (which "cmp")))

It would be better (though more verbose) to use:

  (string-append (assoc-ref inputs "coreutils") "/bin/cmp")

to allow for cross-compilation, though we’d also need to add coreutils
to ‘inputs’.  Not sure if it’s worth it, your call!

Otherwise LGTM.

Thanks,
Ludo’.





reply via email to

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