unofficial mirror of guix-patches@gnu.org 
 help / color / mirror / code / Atom feed
* [bug#66424] [PATCH] gnu: libxkbcommon: Fix cross-compile.
@ 2023-10-09 15:07 Zheng Junjie
  2023-10-12 18:07 ` [bug#66424] [PATCH core-updates] " John Kehayias via Guix-patches via
  0 siblings, 1 reply; 8+ messages in thread
From: Zheng Junjie @ 2023-10-09 15:07 UTC (permalink / raw)
  To: 66424

* gnu/packages/xdisorg.scm (libxkbcommon): Fix cross-compile.
[native-inputs]: When cross-compile add PKG-CONFIG-FOR-BUILD.
---
 gnu/packages/xdisorg.scm | 10 +++++++---
 1 file changed, 7 insertions(+), 3 deletions(-)

diff --git a/gnu/packages/xdisorg.scm b/gnu/packages/xdisorg.scm
index db948a9318..84bdcad031 100644
--- a/gnu/packages/xdisorg.scm
+++ b/gnu/packages/xdisorg.scm
@@ -527,9 +527,13 @@ (define-public libxkbcommon
            wayland-protocols
            xkeyboard-config))
     (native-inputs
-     (list bison doxygen pkg-config python
-           ;; wayland-scanner is required at build time.
-           wayland))
+     (append
+      (if (%current-target-system)
+          (list pkg-config-for-build)
+          '())
+      (list bison doxygen pkg-config python
+            ;; wayland-scanner is required at build time.
+            wayland)))
     (arguments
      (list #:configure-flags
            #~(list (string-append "-Dxkb-config-root="

base-commit: 9b77bd0b9b4f3de69390da0ba7db5b9dbc01e554
-- 
2.41.0





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

* [bug#66424] [PATCH core-updates] gnu: libxkbcommon: Fix cross-compile.
  2023-10-09 15:07 [bug#66424] [PATCH] gnu: libxkbcommon: Fix cross-compile Zheng Junjie
@ 2023-10-12 18:07 ` John Kehayias via Guix-patches via
  2023-10-13 14:49   ` Maxim Cournoyer
  0 siblings, 1 reply; 8+ messages in thread
From: John Kehayias via Guix-patches via @ 2023-10-12 18:07 UTC (permalink / raw)
  To: Zheng Junjie; +Cc: 66424, Maxim Cournoyer, Marius Bakke

Hello,

(CC'ing authors of two related commits too)

On Mon, Oct 09, 2023 at 11:07 PM, Zheng Junjie wrote:

> * gnu/packages/xdisorg.scm (libxkbcommon): Fix cross-compile.
> [native-inputs]: When cross-compile add PKG-CONFIG-FOR-BUILD.
> ---
>  gnu/packages/xdisorg.scm | 10 +++++++---
>  1 file changed, 7 insertions(+), 3 deletions(-)
>
> diff --git a/gnu/packages/xdisorg.scm b/gnu/packages/xdisorg.scm
> index db948a9318..84bdcad031 100644
> --- a/gnu/packages/xdisorg.scm
> +++ b/gnu/packages/xdisorg.scm
> @@ -527,9 +527,13 @@ (define-public libxkbcommon
>             wayland-protocols
>             xkeyboard-config))
>      (native-inputs
> -     (list bison doxygen pkg-config python
> -           ;; wayland-scanner is required at build time.
> -           wayland))
> +     (append
> +      (if (%current-target-system)
> +          (list pkg-config-for-build)
> +          '())
> +      (list bison doxygen pkg-config python
> +            ;; wayland-scanner is required at build time.
> +            wayland)))
>      (arguments
>       (list #:configure-flags
>             #~(list (string-append "-Dxkb-config-root="
>
> base-commit: 9b77bd0b9b4f3de69390da0ba7db5b9dbc01e554

When searching the log it looked like this had already been done on
core-updates in 5841ac3981a3a08968704728cef14accd14e7a1c (Marius) but I
was confused when I didn't see it. Looks like the pkg-config-for-build
was dropped (accidentally?) in 5841ac3981a3a08968704728cef14accd14e7a1c
(Maxim)

Is this still needed?

And if so, I can do it on mesa-updates very soon, since libx11 and
libxpm will be ungrafted and related packages (like mesa) updated too.
Or we can leave it to core-updates if that makes more sense.

WDYT?

Thanks!
John





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

* [bug#66424] [PATCH core-updates] gnu: libxkbcommon: Fix cross-compile.
  2023-10-12 18:07 ` [bug#66424] [PATCH core-updates] " John Kehayias via Guix-patches via
@ 2023-10-13 14:49   ` Maxim Cournoyer
  2023-10-14 17:30     ` zhengjunjie
  0 siblings, 1 reply; 8+ messages in thread
From: Maxim Cournoyer @ 2023-10-13 14:49 UTC (permalink / raw)
  To: John Kehayias; +Cc: Zheng Junjie, 66424, Marius Bakke

Hi John,

John Kehayias <john.kehayias@protonmail.com> writes:

> Hello,
>
> (CC'ing authors of two related commits too)
>
> On Mon, Oct 09, 2023 at 11:07 PM, Zheng Junjie wrote:
>
>> * gnu/packages/xdisorg.scm (libxkbcommon): Fix cross-compile.
>> [native-inputs]: When cross-compile add PKG-CONFIG-FOR-BUILD.
>> ---
>>  gnu/packages/xdisorg.scm | 10 +++++++---
>>  1 file changed, 7 insertions(+), 3 deletions(-)
>>
>> diff --git a/gnu/packages/xdisorg.scm b/gnu/packages/xdisorg.scm
>> index db948a9318..84bdcad031 100644
>> --- a/gnu/packages/xdisorg.scm
>> +++ b/gnu/packages/xdisorg.scm
>> @@ -527,9 +527,13 @@ (define-public libxkbcommon
>>             wayland-protocols
>>             xkeyboard-config))
>>      (native-inputs
>> -     (list bison doxygen pkg-config python
>> -           ;; wayland-scanner is required at build time.
>> -           wayland))
>> +     (append
>> +      (if (%current-target-system)
>> +          (list pkg-config-for-build)
>> +          '())
>> +      (list bison doxygen pkg-config python
>> +            ;; wayland-scanner is required at build time.
>> +            wayland)))
>>      (arguments
>>       (list #:configure-flags
>>             #~(list (string-append "-Dxkb-config-root="
>>
>> base-commit: 9b77bd0b9b4f3de69390da0ba7db5b9dbc01e554
>
> When searching the log it looked like this had already been done on
> core-updates in 5841ac3981a3a08968704728cef14accd14e7a1c (Marius) but I
> was confused when I didn't see it. Looks like the pkg-config-for-build
> was dropped (accidentally?) in 5841ac3981a3a08968704728cef14accd14e7a1c
> (Maxim)
>
> Is this still needed?

On the top of my head, I thought pkg-config-for-build had been obsolete
by the pkg-config symbol being dynamically bound to something that makes
sense for the context it's used in, via some macro trickery.  I may have
misunderstood things.

> And if so, I can do it on mesa-updates very soon, since libx11 and
> libxpm will be ungrafted and related packages (like mesa) updated too.
> Or we can leave it to core-updates if that makes more sense.

I guess it depends which branch is expected to be the shortest lived
one; I'm giving about a month to let more time for core-updates to merge
interesting bits and build the branch and fix breakage. Take this with a
grain of salt; it's not fixed in stone.

Thanks,

Maxim




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

* [bug#66424] [PATCH core-updates] gnu: libxkbcommon: Fix cross-compile.
  2023-10-13 14:49   ` Maxim Cournoyer
@ 2023-10-14 17:30     ` zhengjunjie
  2023-10-14 21:02       ` John Kehayias via Guix-patches via
  2023-10-16  2:28       ` bug#66424: " John Kehayias via Guix-patches via
  0 siblings, 2 replies; 8+ messages in thread
From: zhengjunjie @ 2023-10-14 17:30 UTC (permalink / raw)
  To: Maxim Cournoyer; +Cc: John Kehayias, 66424, Marius Bakke



> 在 2023年10月13日,22:50,Maxim Cournoyer <maxim.cournoyer@gmail.com> 写道:
> 
> 
> On the top of my head, I thought pkg-config-for-build had been obsolete
> by the pkg-config symbol being dynamically bound to something that makes
> sense for the context it's used in, via some macro trickery.  I may have
> misunderstood things

For cross build, 

pkg-config -> xxxx-pkg-config
pkg-config-for-build -> pkg-config

libxkbcommon require wayland-scanner to generate c file, this is why need pkg-config-for-build, pkg-config is require by find library. Is different ,so libxkbcommon need both on cross build.




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

* [bug#66424] [PATCH core-updates] gnu: libxkbcommon: Fix cross-compile.
  2023-10-14 17:30     ` zhengjunjie
@ 2023-10-14 21:02       ` John Kehayias via Guix-patches via
  2023-10-16  2:28       ` bug#66424: " John Kehayias via Guix-patches via
  1 sibling, 0 replies; 8+ messages in thread
From: John Kehayias via Guix-patches via @ 2023-10-14 21:02 UTC (permalink / raw)
  To: zhengjunjie; +Cc: 66424, Maxim Cournoyer, Marius Bakke

Hi,

On Sun, Oct 15, 2023 at 01:30 AM, zhengjunjie@iscas.ac.cn wrote:

>> 在 2023年10月13日,22:50,Maxim Cournoyer <maxim.cournoyer@gmail.com> 写道:
>>
>>
>> On the top of my head, I thought pkg-config-for-build had been obsolete
>> by the pkg-config symbol being dynamically bound to something that makes
>> sense for the context it's used in, via some macro trickery.  I may have
>> misunderstood things
>
> For cross build,
>
> pkg-config -> xxxx-pkg-config
> pkg-config-for-build -> pkg-config
>
> libxkbcommon require wayland-scanner to generate c file, this is why
> need pkg-config-for-build, pkg-config is require by find library. Is
> different ,so libxkbcommon need both on cross build.

I'm not a cross-builder so I'll rely on you all for this.

If this change depends on other things in core-updates I'll leave it,
or if cross-building has had changes on core-updates. If it is just
something that can be done based on current master, I can cherry pick
the change from core-updates and apply this patch on mesa-updates.

What will work for everyone here?

I think my timeline will be quicker but I don't think this is
critical? I was just starting to gather what to apply on the mesa
branch, just waiting for the next mesa release (normally very quick
but current series has been delayed).

Thanks!
John





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

* bug#66424: [PATCH core-updates] gnu: libxkbcommon: Fix cross-compile.
  2023-10-14 17:30     ` zhengjunjie
  2023-10-14 21:02       ` John Kehayias via Guix-patches via
@ 2023-10-16  2:28       ` John Kehayias via Guix-patches via
  2023-10-16  4:42         ` [bug#66424] " Efraim Flashner
  1 sibling, 1 reply; 8+ messages in thread
From: John Kehayias via Guix-patches via @ 2023-10-16  2:28 UTC (permalink / raw)
  To: zhengjunjie; +Cc: Marius Bakke, Efraim Flashner, Maxim Cournoyer, 66424-done

Hi all,

cc'ing Efraim too, since...

On Sun, Oct 15, 2023 at 01:30 AM, zhengjunjie@iscas.ac.cn wrote:

>> 在 2023年10月13日,22:50,Maxim Cournoyer <maxim.cournoyer@gmail.com> 写道:
>>
>>
>> On the top of my head, I thought pkg-config-for-build had been obsolete
>> by the pkg-config symbol being dynamically bound to something that makes
>> sense for the context it's used in, via some macro trickery.  I may have
>> misunderstood things
>
> For cross build,
>
> pkg-config -> xxxx-pkg-config
> pkg-config-for-build -> pkg-config
>
> libxkbcommon require wayland-scanner to generate c file, this is why
> need pkg-config-for-build, pkg-config is require by find library. Is
> different ,so libxkbcommon need both on cross build.

looks like the same change was made on master in
bc1634a197dc9ab7aa0dead8c7449bb5db09cea6. Along with other commits
then to do the same thing to other similar packages.

So, I'm closing as I believe this is done but figured I'd loop in
Efraim in case there was anything else on this matter to discuss.

Thanks all!
John





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

* [bug#66424] [PATCH core-updates] gnu: libxkbcommon: Fix cross-compile.
  2023-10-16  2:28       ` bug#66424: " John Kehayias via Guix-patches via
@ 2023-10-16  4:42         ` Efraim Flashner
  2023-10-16 13:48           ` Maxim Cournoyer
  0 siblings, 1 reply; 8+ messages in thread
From: Efraim Flashner @ 2023-10-16  4:42 UTC (permalink / raw)
  To: John Kehayias; +Cc: Marius Bakke, zhengjunjie, Maxim Cournoyer, 66424-done

[-- Attachment #1: Type: text/plain, Size: 1983 bytes --]

On Mon, Oct 16, 2023 at 02:28:13AM +0000, John Kehayias wrote:
> Hi all,
> 
> cc'ing Efraim too, since...
> 
> On Sun, Oct 15, 2023 at 01:30 AM, zhengjunjie@iscas.ac.cn wrote:
> 
> >> 在 2023年10月13日,22:50,Maxim Cournoyer <maxim.cournoyer@gmail.com> 写道:
> >>
> >>
> >> On the top of my head, I thought pkg-config-for-build had been obsolete
> >> by the pkg-config symbol being dynamically bound to something that makes
> >> sense for the context it's used in, via some macro trickery.  I may have
> >> misunderstood things
> >
> > For cross build,
> >
> > pkg-config -> xxxx-pkg-config
> > pkg-config-for-build -> pkg-config
> >
> > libxkbcommon require wayland-scanner to generate c file, this is why
> > need pkg-config-for-build, pkg-config is require by find library. Is
> > different ,so libxkbcommon need both on cross build.
> 
> looks like the same change was made on master in
> bc1634a197dc9ab7aa0dead8c7449bb5db09cea6. Along with other commits
> then to do the same thing to other similar packages.
> 
> So, I'm closing as I believe this is done but figured I'd loop in
> Efraim in case there was anything else on this matter to discuss.

Oh no! I really need to get better about checking the bug tracker to see
if someone else has already worked through changes I'm thinking of.

I tried a bit of pkg-config hackery/trickery for a bunch of the
packages, even messing with the meson cross definitions file, but it
turned out that pkg-config-for-build did the exact wrapping that was
needed AND it provided a symlink from (pkg-config-for-target) to
"pkg-config", which was what meson needed.

I'll cherry-pick the commits over to core-updates and then do some of
the fixups to make everything cleaner.

-- 
Efraim Flashner   <efraim@flashner.co.il>   רנשלפ םירפא
GPG key = A28B F40C 3E55 1372 662D  14F7 41AA E7DC CA3D 8351
Confidentiality cannot be guaranteed on emails sent or received unencrypted

[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 833 bytes --]

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

* [bug#66424] [PATCH core-updates] gnu: libxkbcommon: Fix cross-compile.
  2023-10-16  4:42         ` [bug#66424] " Efraim Flashner
@ 2023-10-16 13:48           ` Maxim Cournoyer
  0 siblings, 0 replies; 8+ messages in thread
From: Maxim Cournoyer @ 2023-10-16 13:48 UTC (permalink / raw)
  To: Efraim Flashner; +Cc: John Kehayias, Marius Bakke, zhengjunjie, 66424-done

Hi Efraim,

Efraim Flashner <efraim@flashner.co.il> writes:

[...]

>> looks like the same change was made on master in
>> bc1634a197dc9ab7aa0dead8c7449bb5db09cea6. Along with other commits
>> then to do the same thing to other similar packages.
>> 
>> So, I'm closing as I believe this is done but figured I'd loop in
>> Efraim in case there was anything else on this matter to discuss.
>
> Oh no! I really need to get better about checking the bug tracker to see
> if someone else has already worked through changes I'm thinking of.
>
> I tried a bit of pkg-config hackery/trickery for a bunch of the
> packages, even messing with the meson cross definitions file, but it
> turned out that pkg-config-for-build did the exact wrapping that was
> needed AND it provided a symlink from (pkg-config-for-target) to
> "pkg-config", which was what meson needed.
>
> I'll cherry-pick the commits over to core-updates and then do some of
> the fixups to make everything cleaner.

Thanks for the resolution, and sorry for the regression I had
introduced.

-- 
Thanks,
Maxim




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

end of thread, other threads:[~2023-10-16 13:49 UTC | newest]

Thread overview: 8+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2023-10-09 15:07 [bug#66424] [PATCH] gnu: libxkbcommon: Fix cross-compile Zheng Junjie
2023-10-12 18:07 ` [bug#66424] [PATCH core-updates] " John Kehayias via Guix-patches via
2023-10-13 14:49   ` Maxim Cournoyer
2023-10-14 17:30     ` zhengjunjie
2023-10-14 21:02       ` John Kehayias via Guix-patches via
2023-10-16  2:28       ` bug#66424: " John Kehayias via Guix-patches via
2023-10-16  4:42         ` [bug#66424] " Efraim Flashner
2023-10-16 13:48           ` Maxim Cournoyer

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