unofficial mirror of guix-patches@gnu.org 
 help / color / mirror / code / Atom feed
* [bug#64874] [PATCH] guix: tuned-package: Use target on cross-compile.
@ 2023-07-26 15:37 Jean-Pierre De Jesus DIAZ via Guix-patches via
  2023-08-16 16:51 ` Simon Tournier
  0 siblings, 1 reply; 4+ messages in thread
From: Jean-Pierre De Jesus DIAZ via Guix-patches via @ 2023-07-26 15:37 UTC (permalink / raw)
  To: 64874
  Cc: Jean-Pierre De Jesus DIAZ, mail, dev, ludo, othacehe, rekado,
	zimon.toutoune, me

* 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)
                            ((arch _ ...) arch)))
            (compiler     (any (match-lambda
                                 ((label (? package? p) . _)

base-commit: cf9904bcc8dd03e73675475bb4d8746dc434e415
-- 
2.34.1





^ permalink raw reply related	[flat|nested] 4+ messages in thread

* [bug#64874] [PATCH] guix: tuned-package: Use target on cross-compile.
  2023-07-26 15:37 [bug#64874] [PATCH] guix: tuned-package: Use target on cross-compile Jean-Pierre De Jesus DIAZ via Guix-patches via
@ 2023-08-16 16:51 ` Simon Tournier
  2023-08-16 21:54   ` Ludovic Courtès
  0 siblings, 1 reply; 4+ messages in thread
From: Simon Tournier @ 2023-08-16 16:51 UTC (permalink / raw)
  To: 64874; +Cc: dev, me, othacehe, ludo, mail, Jean-Pierre De Jesus DIAZ, rekado

Hi,

Thanks for the patch.


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?

Cheers,
simon





^ permalink raw reply	[flat|nested] 4+ messages in thread

* [bug#64874] [PATCH] guix: tuned-package: Use target on cross-compile.
  2023-08-16 16:51 ` Simon Tournier
@ 2023-08-16 21:54   ` Ludovic Courtès
  2023-09-17 13:08     ` bug#64874: " Ludovic Courtès
  0 siblings, 1 reply; 4+ messages in thread
From: Ludovic Courtès @ 2023-08-16 21:54 UTC (permalink / raw)
  To: Simon Tournier
  Cc: 64874, dev, mail, othacehe, me, Jean-Pierre De Jesus DIAZ, rekado

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’.




^ permalink raw reply	[flat|nested] 4+ messages in thread

* bug#64874: [PATCH] guix: tuned-package: Use target on cross-compile.
  2023-08-16 21:54   ` Ludovic Courtès
@ 2023-09-17 13:08     ` Ludovic Courtès
  0 siblings, 0 replies; 4+ messages in thread
From: Ludovic Courtès @ 2023-09-17 13:08 UTC (permalink / raw)
  To: Simon Tournier
  Cc: dev, me, othacehe, mail, Jean-Pierre De Jesus DIAZ, rekado,
	64874-done

Hi,

Ludovic Courtès <ludo@gnu.org> skribis:

> Simon Tournier <zimon.toutoune@gmail.com> skribis:

[...]

>> Ludo, WDYT?
>
> LGTM, thanks for the heads-up!

I went ahead and applied it.

Ludo’.




^ permalink raw reply	[flat|nested] 4+ messages in thread

end of thread, other threads:[~2023-09-17 13:09 UTC | newest]

Thread overview: 4+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2023-07-26 15:37 [bug#64874] [PATCH] guix: tuned-package: Use target on cross-compile Jean-Pierre De Jesus DIAZ via Guix-patches via
2023-08-16 16:51 ` Simon Tournier
2023-08-16 21:54   ` Ludovic Courtès
2023-09-17 13:08     ` bug#64874: " Ludovic Courtès

Code repositories for project(s) associated with this public inbox

	https://git.savannah.gnu.org/cgit/guix.git

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).