[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[bug#64874] [PATCH] guix: tuned-package: Use target on cross-compile.
From: |
Ludovic Courtès |
Subject: |
[bug#64874] [PATCH] guix: tuned-package: Use target on cross-compile. |
Date: |
Wed, 16 Aug 2023 23:54:01 +0200 |
User-agent: |
Gnus/5.13 (Gnus v5.13) Emacs/28.2 (gnu/linux) |
Hi,
Simon Tournier <zimon.toutoune@gmail.com> skribis:
> On Wed, 26 Jul 2023 at 17:37, Jean-Pierre De Jesus DIAZ via Guix-patches via
> <guix-patches@gnu.org> wrote:
>> * guix/transformations.scm (tuned-package): Use either bag-target if
>> available or bag-system to select the CPU architecture of the package
>> that is going to be tuned. This enables the tuning of cross-compiled
>> packages.
>>
>> Signed-off-by: Jean-Pierre De Jesus DIAZ <jean@foundationdevices.com>
>> ---
>> guix/transformations.scm | 5 +++--
>> 1 file changed, 3 insertions(+), 2 deletions(-)
>>
>> diff --git a/guix/transformations.scm b/guix/transformations.scm
>> index 92d9c89c0e..76d0208bf1 100644
>> --- a/guix/transformations.scm
>> +++ b/guix/transformations.scm
>> @@ -510,8 +510,9 @@ (define (lower* . args)
>> ;; leading to an obscure build error, check whether the compiler is
>> known
>> ;; to support MICRO-ARCHITECTURE. If not, bail out.
>> (let* ((lowered (apply lower args))
>> - (architecture (match (string-tokenize (bag-system lowered)
>> - %not-hyphen)
>> + (target (or (bag-target lowered)
>> + (bag-system lowered)))
>> + (architecture (match (string-tokenize target %not-hyphen)
>
> Ludo, WDYT?
LGTM, thanks for the heads-up!
Ludo’.