[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!
signature.asc
Description: PGP signature