unofficial mirror of guix-patches@gnu.org 
 help / color / mirror / code / Atom feed
* [bug#51953] [PATCH core-updates-frozen 0/2] Fix wlroots build with Meson 0.60.0
@ 2021-11-18 16:43 Jack Hill
  2021-11-18 16:44 ` [bug#51953] [PATCH core-updates-frozen 1/2] gnu: wlroots: Remove logind-provider configure option Jack Hill
  2021-11-18 17:22 ` [bug#51953] [PATCH core-updates-frozen 0/2] Fix wlroots build with Meson 0.60.0 Josselin Poiret via Guix-patches via
  0 siblings, 2 replies; 5+ messages in thread
From: Jack Hill @ 2021-11-18 16:43 UTC (permalink / raw)
  To: 51953

The build system now automatically detects which logind to use [0], and 
specifying the option is an error with Meson 0.60.0.

[0] https://gitlab.freedesktop.org/wlroots/wlroots/-/commit/30308e35fac0491ab777495e2ee7e472378d375a

While I was modifying the package, I also updated its home-page to point 
to the new freedesktop.org location.

Jack Hill (2):
   gnu: wlroots: Remove logind-provider configure option.
   gnu: wlroots: Update home page.

  gnu/packages/wm.scm | 5 ++---
  1 file changed, 2 insertions(+), 3 deletions(-)

-- 
2.33.1




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

* [bug#51953] [PATCH core-updates-frozen 1/2] gnu: wlroots: Remove logind-provider configure option.
  2021-11-18 16:43 [bug#51953] [PATCH core-updates-frozen 0/2] Fix wlroots build with Meson 0.60.0 Jack Hill
@ 2021-11-18 16:44 ` Jack Hill
  2021-11-18 16:44   ` [bug#51953] [PATCH core-updates-frozen 2/2] gnu: wlroots: Update home page Jack Hill
  2021-11-18 17:22 ` [bug#51953] [PATCH core-updates-frozen 0/2] Fix wlroots build with Meson 0.60.0 Josselin Poiret via Guix-patches via
  1 sibling, 1 reply; 5+ messages in thread
From: Jack Hill @ 2021-11-18 16:44 UTC (permalink / raw)
  To: 51953

* gnu/packages/wm.scm (wlroots)[arguments]: Remove #:configure-flags.
---
 gnu/packages/wm.scm | 3 +--
 1 file changed, 1 insertion(+), 2 deletions(-)

diff --git a/gnu/packages/wm.scm b/gnu/packages/wm.scm
index 7b7d04eb93..d3edfe3328 100644
--- a/gnu/packages/wm.scm
+++ b/gnu/packages/wm.scm
@@ -1480,8 +1480,7 @@ (define-public wlroots
         (base32 "1sshp3lvlkl1i670kxhwsb4xzxl8raz6769kqvgmxzcb63ns9ay1"))))
     (build-system meson-build-system)
     (arguments
-     `(#:configure-flags '("-Dlogind-provider=elogind")
-       #:phases
+     '(#:phases
        (modify-phases %standard-phases
          (add-before 'configure 'hardcode-paths
            (lambda* (#:key inputs #:allow-other-keys)
-- 
2.33.1





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

* [bug#51953] [PATCH core-updates-frozen 2/2] gnu: wlroots: Update home page.
  2021-11-18 16:44 ` [bug#51953] [PATCH core-updates-frozen 1/2] gnu: wlroots: Remove logind-provider configure option Jack Hill
@ 2021-11-18 16:44   ` Jack Hill
  0 siblings, 0 replies; 5+ messages in thread
From: Jack Hill @ 2021-11-18 16:44 UTC (permalink / raw)
  To: 51953

* gnu/packages/wm.scm (wlroots)[home-page]: Update it.
---
 gnu/packages/wm.scm | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/gnu/packages/wm.scm b/gnu/packages/wm.scm
index d3edfe3328..a9e13d1cd8 100644
--- a/gnu/packages/wm.scm
+++ b/gnu/packages/wm.scm
@@ -1505,7 +1505,7 @@ (define-public wlroots
        ("xorg-server-xwayland" ,xorg-server-xwayland)))
     (native-inputs
      `(("pkg-config" ,pkg-config)))
-    (home-page "https://github.com/swaywm/wlroots")
+    (home-page "https://gitlab.freedesktop.org/wlroots/wlroots")
     (synopsis "Pluggable, composable, unopinionated modules for building a
 Wayland compositor")
     (description "wlroots is a set of pluggable, composable, unopinionated
-- 
2.33.1





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

* [bug#51953] [PATCH core-updates-frozen 0/2] Fix wlroots build with Meson 0.60.0
  2021-11-18 16:43 [bug#51953] [PATCH core-updates-frozen 0/2] Fix wlroots build with Meson 0.60.0 Jack Hill
  2021-11-18 16:44 ` [bug#51953] [PATCH core-updates-frozen 1/2] gnu: wlroots: Remove logind-provider configure option Jack Hill
@ 2021-11-18 17:22 ` Josselin Poiret via Guix-patches via
  2021-11-19 15:52   ` bug#51953: " Jack Hill
  1 sibling, 1 reply; 5+ messages in thread
From: Josselin Poiret via Guix-patches via @ 2021-11-18 17:22 UTC (permalink / raw)
  To: Jack Hill, 51953; +Cc: dev

Jack Hill <jackhill@jackhill.us> writes:

> The build system now automatically detects which logind to use [0], and 
> specifying the option is an error with Meson 0.60.0.
>
> [0] https://gitlab.freedesktop.org/wlroots/wlroots/-/commit/30308e35fac0491ab777495e2ee7e472378d375a

elogind is no longer a dependency of wlroots because of
https://gitlab.freedesktop.org/wlroots/wlroots/-/commit/d50bbf0bbc49a9a8e010668cd357aba968ae884b
included in 0.14.

It is a dependency of seatd though, and it should be propagated because
it is required by the pkg-config of seatd.  Since you still specified
elogind here, pkg-config was able to find it nonetheless but this isn't
the proper way to do it.

Best,
Josselin Poiret




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

* bug#51953: [PATCH core-updates-frozen 0/2] Fix wlroots build with Meson 0.60.0
  2021-11-18 17:22 ` [bug#51953] [PATCH core-updates-frozen 0/2] Fix wlroots build with Meson 0.60.0 Josselin Poiret via Guix-patches via
@ 2021-11-19 15:52   ` Jack Hill
  0 siblings, 0 replies; 5+ messages in thread
From: Jack Hill @ 2021-11-19 15:52 UTC (permalink / raw)
  To: Josselin Poiret; +Cc: 51953-done

On Thu, 18 Nov 2021, Josselin Poiret via Guix-patches via wrote:

> Jack Hill <jackhill@jackhill.us> writes:
>
>> The build system now automatically detects which logind to use [0], and
>> specifying the option is an error with Meson 0.60.0.
>>
>> [0] https://gitlab.freedesktop.org/wlroots/wlroots/-/commit/30308e35fac0491ab777495e2ee7e472378d375a
>
> elogind is no longer a dependency of wlroots because of
> https://gitlab.freedesktop.org/wlroots/wlroots/-/commit/d50bbf0bbc49a9a8e010668cd357aba968ae884b
> included in 0.14.
>
> It is a dependency of seatd though, and it should be propagated because
> it is required by the pkg-config of seatd.  Since you still specified
> elogind here, pkg-config was able to find it nonetheless but this isn't
> the proper way to do it.

Ah, thanks for pointing that out, and submitted the proper fix in #51972.

I'm closing this issue.

Best,
Jack




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

end of thread, other threads:[~2021-11-19 15:53 UTC | newest]

Thread overview: 5+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2021-11-18 16:43 [bug#51953] [PATCH core-updates-frozen 0/2] Fix wlroots build with Meson 0.60.0 Jack Hill
2021-11-18 16:44 ` [bug#51953] [PATCH core-updates-frozen 1/2] gnu: wlroots: Remove logind-provider configure option Jack Hill
2021-11-18 16:44   ` [bug#51953] [PATCH core-updates-frozen 2/2] gnu: wlroots: Update home page Jack Hill
2021-11-18 17:22 ` [bug#51953] [PATCH core-updates-frozen 0/2] Fix wlroots build with Meson 0.60.0 Josselin Poiret via Guix-patches via
2021-11-19 15:52   ` bug#51953: " Jack Hill

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