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: Marius Bakke
Subject: bug#30013: [PATCH] gnu: Add vboot-reference.
Date: Mon, 08 Jan 2018 20:14:35 +0100
User-agent: Notmuch/0.25.3 (https://notmuchmail.org) Emacs/25.3.1 (x86_64-pc-linux-gnu)

Ludovic Courtès <address@hidden> writes:

> Hi!
>
> Marius Bakke <address@hidden> skribis:
>
>> * gnu/packages/bootloaders.scm (vboot-reference): New public variable.
>
> “vboot-utils”

Whoops, good catch.

>> +       #: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!

Thanks for this tip.  I packaged this for use with an ARM Chromebook, so
facilitating cross-compilation is good.  Updated!

Thanks for reviewing!

Attachment: signature.asc
Description: PGP signature


reply via email to

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