unofficial mirror of guix-patches@gnu.org 
 help / color / mirror / code / Atom feed
* [bug#36386] [PATCH] gnu: mesa: Fix detection of wayland-scanner pkg-config file.
@ 2019-06-26  1:53 Kei Kebreau
  2019-06-26  1:53 ` [bug#36387] " Kei Kebreau
  2019-06-30 15:10 ` bug#36386: " Marius Bakke
  0 siblings, 2 replies; 6+ messages in thread
From: Kei Kebreau @ 2019-06-26  1:53 UTC (permalink / raw)
  To: 36386

This is a patch that enables mesa to build on my machine on the core-updates
branch as of June 25. Somehow, meson (the build system, not "mesa") fails to
detect wayland-scanner when the keyword argument "native" is set to true.
Hopefully this change does not cause trouble with cross-building.

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

* [bug#36387] [PATCH] gnu: mesa: Fix detection of wayland-scanner pkg-config file.
  2019-06-26  1:53 [bug#36386] [PATCH] gnu: mesa: Fix detection of wayland-scanner pkg-config file Kei Kebreau
@ 2019-06-26  1:53 ` Kei Kebreau
  2021-11-14  1:22   ` Maxim Cournoyer
  2019-06-30 15:10 ` bug#36386: " Marius Bakke
  1 sibling, 1 reply; 6+ messages in thread
From: Kei Kebreau @ 2019-06-26  1:53 UTC (permalink / raw)
  To: 36387; +Cc: Kei Kebreau

* gnu/packages/gl.scm (mesa)[arguments]: Add "patch-meson-build" phase.
---
 gnu/packages/gl.scm | 9 +++++++++
 1 file changed, 9 insertions(+)

diff --git a/gnu/packages/gl.scm b/gnu/packages/gl.scm
index 96e7fc7165..78f7401f20 100644
--- a/gnu/packages/gl.scm
+++ b/gnu/packages/gl.scm
@@ -337,6 +337,15 @@ also known as DXTn or DXTC) for Mesa.")
                        (("'u_format_test',") ""))
                      #t)))
                '())
+         (add-after
+           'unpack 'patch-meson-build
+           (lambda _
+             ;; Without this, meson fails with the following error:
+             ;; Dependency "wayland-scanner" not found, tried pkgconfig.
+             (substitute* "meson.build"
+               (("dependency\\('wayland-scanner', native: true\\)")
+                "dependency('wayland-scanner')"))
+             #t))
          (add-before
            'configure 'fix-dlopen-libnames
            (lambda* (#:key inputs outputs #:allow-other-keys)
-- 
2.22.0

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

* bug#36386: [PATCH] gnu: mesa: Fix detection of wayland-scanner pkg-config file.
  2019-06-26  1:53 [bug#36386] [PATCH] gnu: mesa: Fix detection of wayland-scanner pkg-config file Kei Kebreau
  2019-06-26  1:53 ` [bug#36387] " Kei Kebreau
@ 2019-06-30 15:10 ` Marius Bakke
  1 sibling, 0 replies; 6+ messages in thread
From: Marius Bakke @ 2019-06-30 15:10 UTC (permalink / raw)
  To: Kei Kebreau, 36386-done

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

Kei Kebreau <kkebreau@posteo.net> writes:

> This is a patch that enables mesa to build on my machine on the core-updates
> branch as of June 25. Somehow, meson (the build system, not "mesa") fails to
> detect wayland-scanner when the keyword argument "native" is set to true.
> Hopefully this change does not cause trouble with cross-building.

This is due to a regression in Meson 0.51:

<https://github.com/mesonbuild/meson/issues/5502>

I reverted the upgrade for now, in commit
fd3594bf3881fb8d4c04f76db0f1a9cbd5fae871.  Thanks!

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

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

* [bug#36387] [PATCH] gnu: mesa: Fix detection of wayland-scanner pkg-config file.
  2019-06-26  1:53 ` [bug#36387] " Kei Kebreau
@ 2021-11-14  1:22   ` Maxim Cournoyer
  2021-12-27 13:46     ` Kei Kebreau
  0 siblings, 1 reply; 6+ messages in thread
From: Maxim Cournoyer @ 2021-11-14  1:22 UTC (permalink / raw)
  To: Kei Kebreau; +Cc: 36387

Hello,

Kei Kebreau <kkebreau@posteo.net> writes:

> * gnu/packages/gl.scm (mesa)[arguments]: Add "patch-meson-build" phase.
> ---
>  gnu/packages/gl.scm | 9 +++++++++
>  1 file changed, 9 insertions(+)
>
> diff --git a/gnu/packages/gl.scm b/gnu/packages/gl.scm
> index 96e7fc7165..78f7401f20 100644
> --- a/gnu/packages/gl.scm
> +++ b/gnu/packages/gl.scm
> @@ -337,6 +337,15 @@ also known as DXTn or DXTC) for Mesa.")
>                         (("'u_format_test',") ""))
>                       #t)))
>                 '())
> +         (add-after
> +           'unpack 'patch-meson-build
> +           (lambda _
> +             ;; Without this, meson fails with the following error:
> +             ;; Dependency "wayland-scanner" not found, tried pkgconfig.
> +             (substitute* "meson.build"
> +               (("dependency\\('wayland-scanner', native: true\\)")
> +                "dependency('wayland-scanner')"))
> +             #t))
>           (add-before
>             'configure 'fix-dlopen-libnames
>             (lambda* (#:key inputs outputs #:allow-other-keys)

Was this fixing a build failure of mesa itself?  Or does it manifest
when building other packages depending on mesa?

Given the patch is 2 years old, I'm guessing it went stale?

Thanks,

Maxim




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

* [bug#36387] [PATCH] gnu: mesa: Fix detection of wayland-scanner pkg-config file.
  2021-11-14  1:22   ` Maxim Cournoyer
@ 2021-12-27 13:46     ` Kei Kebreau
  2021-12-28  2:34       ` bug#36387: " Maxim Cournoyer
  0 siblings, 1 reply; 6+ messages in thread
From: Kei Kebreau @ 2021-12-27 13:46 UTC (permalink / raw)
  To: Maxim Cournoyer; +Cc: 36387

It did in fact go stale. I vaguely remember having some trouble building 
mesa locally, but this no longer seems to be the case.

On 11/13/21 8:22 PM, Maxim Cournoyer wrote:
> Hello,
>
> Kei Kebreau<kkebreau@posteo.net>  writes:
>
>> * gnu/packages/gl.scm (mesa)[arguments]: Add "patch-meson-build" phase.
>> ---
>>   gnu/packages/gl.scm | 9 +++++++++
>>   1 file changed, 9 insertions(+)
>>
>> diff --git a/gnu/packages/gl.scm b/gnu/packages/gl.scm
>> index 96e7fc7165..78f7401f20 100644
>> --- a/gnu/packages/gl.scm
>> +++ b/gnu/packages/gl.scm
>> @@ -337,6 +337,15 @@ also known as DXTn or DXTC) for Mesa.")
>>                          (("'u_format_test',") ""))
>>                        #t)))
>>                  '())
>> +         (add-after
>> +           'unpack 'patch-meson-build
>> +           (lambda _
>> +             ;; Without this, meson fails with the following error:
>> +             ;; Dependency "wayland-scanner" not found, tried pkgconfig.
>> +             (substitute* "meson.build"
>> +               (("dependency\\('wayland-scanner', native: true\\)")
>> +                "dependency('wayland-scanner')"))
>> +             #t))
>>            (add-before
>>              'configure 'fix-dlopen-libnames
>>              (lambda* (#:key inputs outputs #:allow-other-keys)
> Was this fixing a build failure of mesa itself?  Or does it manifest
> when building other packages depending on mesa?
>
> Given the patch is 2 years old, I'm guessing it went stale?
>
> Thanks,
>
> Maxim




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

* bug#36387: [PATCH] gnu: mesa: Fix detection of wayland-scanner pkg-config file.
  2021-12-27 13:46     ` Kei Kebreau
@ 2021-12-28  2:34       ` Maxim Cournoyer
  0 siblings, 0 replies; 6+ messages in thread
From: Maxim Cournoyer @ 2021-12-28  2:34 UTC (permalink / raw)
  To: Kei Kebreau; +Cc: 36387-done

Hello,

Kei Kebreau <kkebreau@posteo.net> writes:

> It did in fact go stale. I vaguely remember having some trouble
> building mesa locally, but this no longer seems to be the case.

Alright, thank you for the heads up.  Closing.

Maxim




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

end of thread, other threads:[~2021-12-28  2:35 UTC | newest]

Thread overview: 6+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2019-06-26  1:53 [bug#36386] [PATCH] gnu: mesa: Fix detection of wayland-scanner pkg-config file Kei Kebreau
2019-06-26  1:53 ` [bug#36387] " Kei Kebreau
2021-11-14  1:22   ` Maxim Cournoyer
2021-12-27 13:46     ` Kei Kebreau
2021-12-28  2:34       ` bug#36387: " Maxim Cournoyer
2019-06-30 15:10 ` bug#36386: " Marius Bakke

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