unofficial mirror of guix-patches@gnu.org 
 help / color / mirror / code / Atom feed
* [bug#63645] [PATCH] gnu: swaynotificationcenter: Update to 0.8.0
@ 2023-05-22 14:20 Brian Cully via Guix-patches via
  2023-06-12 18:38 ` Christopher Baines
  2023-06-16 23:52 ` [bug#63645] [PATCH v2] " Brian Cully via Guix-patches via
  0 siblings, 2 replies; 6+ messages in thread
From: Brian Cully via Guix-patches via @ 2023-05-22 14:20 UTC (permalink / raw)
  To: 63645; +Cc: Brian Cully

* gnu/packages/wm.scm (swaynotificationcenter): Update to 0.8.0.
---
 gnu/packages/wm.scm | 11 ++++++++---
 1 file changed, 8 insertions(+), 3 deletions(-)

diff --git a/gnu/packages/wm.scm b/gnu/packages/wm.scm
index 4360f3722b..1bf6d46200 100644
--- a/gnu/packages/wm.scm
+++ b/gnu/packages/wm.scm
@@ -147,6 +147,7 @@ (define-module (gnu packages wm)
   #:use-module (gnu packages python-xyz)
   #:use-module (gnu packages readline)
   #:use-module (gnu packages serialization)
+  #:use-module (gnu packages shells)
   #:use-module (gnu packages sphinx)
   #:use-module (gnu packages suckless)
   #:use-module (gnu packages texinfo)
@@ -1761,7 +1762,7 @@ (define-public swaybg
 (define-public swaynotificationcenter
   (package
     (name "swaynotificationcenter")
-    (version "0.7.3")
+    (version "0.8.0")
     (source (origin
               (method git-fetch)
               (uri (git-reference
@@ -1769,7 +1770,7 @@ (define-public swaynotificationcenter
                     (commit (string-append "v" version))))
               (file-name (git-file-name name version))
               (sha256
-               (base32 "1xvr5m5sqznr3dd512i5pk0d56v7n0ywdcy6rnz85vbf2k7b6kj5"))))
+               (base32 "1c3gd6mlr209jzzrh5jmws2lawnn3gr6smvzcw74kkpi3wvs7l0k"))))
     (build-system meson-build-system)
     (arguments (list #:configure-flags #~(list "-Dsystemd-service=false")))
     (native-inputs
@@ -1780,11 +1781,15 @@ (define-public swaynotificationcenter
            scdoc
            vala))
     (inputs
-     (list json-glib
+     (list bash-completion
+           fish
+           json-glib
            glib
            gtk+
            gtk-layer-shell
            libhandy
+           libgee
+           pulseaudio
            wayland-protocols))
     (synopsis "Notification daemon with a graphical interface")
     (description

base-commit: 849286ba66c96534bddc04df1a47d5692cbc977e
prerequisite-patch-id: 18557da18beab5677060ea1de1897768791ce89e
prerequisite-patch-id: a2d8b917b767b5445112f49ca4d9b060e8baa587
prerequisite-patch-id: eabe444a8c70abebec53c814fbd88236bb86fabf
-- 
2.40.1





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

* [bug#63645] [PATCH] gnu: swaynotificationcenter: Update to 0.8.0
  2023-05-22 14:20 [bug#63645] [PATCH] gnu: swaynotificationcenter: Update to 0.8.0 Brian Cully via Guix-patches via
@ 2023-06-12 18:38 ` Christopher Baines
  2023-06-12 19:11   ` Brian Cully via Guix-patches via
  2023-06-16 23:52 ` [bug#63645] [PATCH v2] " Brian Cully via Guix-patches via
  1 sibling, 1 reply; 6+ messages in thread
From: Christopher Baines @ 2023-06-12 18:38 UTC (permalink / raw)
  To: Brian Cully; +Cc: 63645

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


Brian Cully via Guix-patches via <guix-patches@gnu.org> writes:

> * gnu/packages/wm.scm (swaynotificationcenter): Update to 0.8.0.
> ---
>  gnu/packages/wm.scm | 11 ++++++++---
>  1 file changed, 8 insertions(+), 3 deletions(-)
>
> diff --git a/gnu/packages/wm.scm b/gnu/packages/wm.scm
> index 4360f3722b..1bf6d46200 100644
> --- a/gnu/packages/wm.scm
> +++ b/gnu/packages/wm.scm
> @@ -147,6 +147,7 @@ (define-module (gnu packages wm)
>    #:use-module (gnu packages python-xyz)
>    #:use-module (gnu packages readline)
>    #:use-module (gnu packages serialization)
> +  #:use-module (gnu packages shells)
>    #:use-module (gnu packages sphinx)
>    #:use-module (gnu packages suckless)
>    #:use-module (gnu packages texinfo)
> @@ -1761,7 +1762,7 @@ (define-public swaybg
>  (define-public swaynotificationcenter
>    (package
>      (name "swaynotificationcenter")
> -    (version "0.7.3")
> +    (version "0.8.0")
>      (source (origin
>                (method git-fetch)
>                (uri (git-reference
> @@ -1769,7 +1770,7 @@ (define-public swaynotificationcenter
>                      (commit (string-append "v" version))))
>                (file-name (git-file-name name version))
>                (sha256
> -               (base32 "1xvr5m5sqznr3dd512i5pk0d56v7n0ywdcy6rnz85vbf2k7b6kj5"))))
> +               (base32 "1c3gd6mlr209jzzrh5jmws2lawnn3gr6smvzcw74kkpi3wvs7l0k"))))
>      (build-system meson-build-system)
>      (arguments (list #:configure-flags #~(list "-Dsystemd-service=false")))
>      (native-inputs
> @@ -1780,11 +1781,15 @@ (define-public swaynotificationcenter
>             scdoc
>             vala))
>      (inputs
> -     (list json-glib
> +     (list bash-completion
> +           fish
> +           json-glib
>             glib
>             gtk+
>             gtk-layer-shell
>             libhandy
> +           libgee
> +           pulseaudio
>             wayland-protocols))

Hi Brian,

Regarding these new inputs, what's you're reasoning for adding them?

fish stood out to me, it doesn't seem to be referenced by the
swaynotificationcenter output (doesn't show up in guix size), so maybe
it should be a native-input if it's some test dependency?

Thanks,

Chris

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

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

* [bug#63645] [PATCH] gnu: swaynotificationcenter: Update to 0.8.0
  2023-06-12 18:38 ` Christopher Baines
@ 2023-06-12 19:11   ` Brian Cully via Guix-patches via
  2023-06-16 23:12     ` Brian Cully via Guix-patches via
  0 siblings, 1 reply; 6+ messages in thread
From: Brian Cully via Guix-patches via @ 2023-06-12 19:11 UTC (permalink / raw)
  To: Christopher Baines; +Cc: 63645


Christopher Baines <mail@cbaines.net> writes:

>>      (inputs
>> -     (list json-glib
>> +     (list bash-completion
>> +           fish
>> +           json-glib
>>             glib
>>             gtk+
>>             gtk-layer-shell
>>             libhandy
>> +           libgee
>> +           pulseaudio
>>             wayland-protocols))
>
> Regarding these new inputs, what's you're reasoning for adding 
> them?

In general: so that configure would find everything it was looking 
for. I believe the only new hard requirement is libgee and 
json-glib (I can test a build without them and see how far I get, 
but it'll have to wait until I have a bit more time).

The others — bash-completion and fish — are for command 
completion, I believe. Do those better belong in native-inputs 
just so the can be detected during configure-time?

-bjc




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

* [bug#63645] [PATCH] gnu: swaynotificationcenter: Update to 0.8.0
  2023-06-12 19:11   ` Brian Cully via Guix-patches via
@ 2023-06-16 23:12     ` Brian Cully via Guix-patches via
  0 siblings, 0 replies; 6+ messages in thread
From: Brian Cully via Guix-patches via @ 2023-06-16 23:12 UTC (permalink / raw)
  To: Brian Cully; +Cc: Christopher Baines, 63645


Brian Cully <bjc@spork.org> writes:

> Christopher Baines <mail@cbaines.net> writes:
>
>>>      (inputs
>>> -     (list json-glib
>>> +     (list bash-completion
>>> +           fish
>>> +           json-glib
>>>             glib
>>>             gtk+
>>>             gtk-layer-shell
>>>             libhandy
>>> +           libgee
>>> +           pulseaudio
>>>             wayland-protocols))
>>
>> Regarding these new inputs, what's you're reasoning for adding 
>> them?
>
> In general: so that configure would find everything it was 
> looking
> for. I believe the only new hard requirement is libgee and 
> json-glib
> (I can test a build without them and see how far I get, but 
> it'll have
> to wait until I have a bit more time).

I re-ran the build today, and the output from the configure phase 
does specify that ‘libgee’, ‘json-glib’, and ‘pulseaudio’ are 
runtime dependencies:

--8<---------------cut here---------------start------------->8---
Run-time dependency json-glib-1.0 found: YES 1.6.2
[…]
Run-time dependency gee-0.8 found: YES 0.20.5
Run-time dependency libpulse found: YES 16.1
Run-time dependency libpulse-mainloop-glib found: YES 16.1
--8<---------------cut here---------------end--------------->8---


> The others — bash-completion and fish — are for command 
> completion, I
> believe. Do those better belong in native-inputs just so the can 
> be
> detected during configure-time?

But maybe ‘Run-time dependency’ is a mis-labelling. Compiling 
without the ‘bash-completion’ or ‘fish’ inputs generates the 
complaints:

--8<---------------cut here---------------start------------->8---
Run-time dependency bash-completion found: NO (tried pkgconfig and 
cmake)
Run-time dependency fish found: NO (tried pkgconfig and cmake)
--8<---------------cut here---------------end--------------->8---

But it still installs the completion files:

--8<---------------cut here---------------start------------->8---
Installing 
/tmp/guix-build-swaynotificationcenter-0.8.0.drv-0/source/completions/zsh/_swaync 
to 
/gnu/store/5v2p9n53ivj7cmq7m442gy6yvzqn9vbz-swaynotificationcenter-0.8.0/share/zsh/site-functions
Installing 
/tmp/guix-build-swaynotificationcenter-0.8.0.drv-0/source/completions/zsh/_swaync-client 
to 
/gnu/store/5v2p9n53ivj7cmq7m442gy6yvzqn9vbz-swaynotificationcenter-0.8.0/share/zsh/site-functions
Installing 
/tmp/guix-build-swaynotificationcenter-0.8.0.drv-0/source/completions/bash/swaync 
to 
/gnu/store/5v2p9n53ivj7cmq7m442gy6yvzqn9vbz-swaynotificationcenter-0.8.0/share/bash-completion/completions
Installing 
/tmp/guix-build-swaynotificationcenter-0.8.0.drv-0/source/completions/bash/swaync-client 
to 
/gnu/store/5v2p9n53ivj7cmq7m442gy6yvzqn9vbz-swaynotificationcenter-0.8.0/share/bash-completion/completions
Installing 
/tmp/guix-build-swaynotificationcenter-0.8.0.drv-0/source/completions/fish/swaync.fish 
to 
/gnu/store/5v2p9n53ivj7cmq7m442gy6yvzqn9vbz-swaynotificationcenter-0.8.0/share/fish/vendor_completions.d
Installing 
/tmp/guix-build-swaynotificationcenter-0.8.0.drv-0/source/completions/fish/swaync-client.fish 
to 
/gnu/store/5v2p9n53ivj7cmq7m442gy6yvzqn9vbz-swaynotificationcenter-0.8.0/share/fish/vendor_completions.d
--8<---------------cut here---------------end--------------->8---

Note also that ‘zsh’ gets completions as well, even though it's 
not being checked for in the configuration phase.

Starting a shell with fish:

--8<---------------cut here---------------start------------->8---
/pre-inst-env guix shell -CP coreutils fish bash 
swaynotificationcenter -- fish
--8<---------------cut here---------------end--------------->8---

Yields completion candidates for ‘swaync’ and ‘swaync-client’, 
even though I've removed those inputs, so it seems they're not 
necessary.

I'll submit a v2 without the ‘fish’ and ‘bash’ inputs shortly.

-- 
-bjc




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

* [bug#63645] [PATCH v2] gnu: swaynotificationcenter: Update to 0.8.0
  2023-05-22 14:20 [bug#63645] [PATCH] gnu: swaynotificationcenter: Update to 0.8.0 Brian Cully via Guix-patches via
  2023-06-12 18:38 ` Christopher Baines
@ 2023-06-16 23:52 ` Brian Cully via Guix-patches via
  2023-06-19 10:14   ` bug#63645: " Christopher Baines
  1 sibling, 1 reply; 6+ messages in thread
From: Brian Cully via Guix-patches via @ 2023-06-16 23:52 UTC (permalink / raw)
  To: 63645; +Cc: Brian Cully

* gnu/packages/wm.scm (swaynotificationcenter): Update to 0.8.0.
---
 gnu/packages/wm.scm | 6 ++++--
 1 file changed, 4 insertions(+), 2 deletions(-)

diff --git a/gnu/packages/wm.scm b/gnu/packages/wm.scm
index e707a15785..55c7e7e63f 100644
--- a/gnu/packages/wm.scm
+++ b/gnu/packages/wm.scm
@@ -1779,7 +1779,7 @@ (define-public swaybg
 (define-public swaynotificationcenter
   (package
     (name "swaynotificationcenter")
-    (version "0.7.3")
+    (version "0.8.0")
     (source (origin
               (method git-fetch)
               (uri (git-reference
@@ -1787,7 +1787,7 @@ (define-public swaynotificationcenter
                     (commit (string-append "v" version))))
               (file-name (git-file-name name version))
               (sha256
-               (base32 "1xvr5m5sqznr3dd512i5pk0d56v7n0ywdcy6rnz85vbf2k7b6kj5"))))
+               (base32 "1c3gd6mlr209jzzrh5jmws2lawnn3gr6smvzcw74kkpi3wvs7l0k"))))
     (build-system meson-build-system)
     (arguments (list #:configure-flags #~(list "-Dsystemd-service=false")))
     (native-inputs
@@ -1803,6 +1803,8 @@ (define-public swaynotificationcenter
            gtk+
            gtk-layer-shell
            libhandy
+           libgee
+           pulseaudio
            wayland-protocols))
     (synopsis "Notification daemon with a graphical interface")
     (description

base-commit: 31336e9f5d68512a9c1c6826bce9f17c892a2125
-- 
2.40.1





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

* bug#63645: [PATCH v2] gnu: swaynotificationcenter: Update to 0.8.0
  2023-06-16 23:52 ` [bug#63645] [PATCH v2] " Brian Cully via Guix-patches via
@ 2023-06-19 10:14   ` Christopher Baines
  0 siblings, 0 replies; 6+ messages in thread
From: Christopher Baines @ 2023-06-19 10:14 UTC (permalink / raw)
  To: Brian Cully; +Cc: 63645-done

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


Brian Cully via Guix-patches via <guix-patches@gnu.org> writes:

> * gnu/packages/wm.scm (swaynotificationcenter): Update to 0.8.0.
> ---
>  gnu/packages/wm.scm | 6 ++++--
>  1 file changed, 4 insertions(+), 2 deletions(-)

Thanks for investigating further Brian, I've pushed this to master as
c63e6652861d8ed85ffa91d568e69053478a1511.

Thanks,

Chris

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

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

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

Thread overview: 6+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2023-05-22 14:20 [bug#63645] [PATCH] gnu: swaynotificationcenter: Update to 0.8.0 Brian Cully via Guix-patches via
2023-06-12 18:38 ` Christopher Baines
2023-06-12 19:11   ` Brian Cully via Guix-patches via
2023-06-16 23:12     ` Brian Cully via Guix-patches via
2023-06-16 23:52 ` [bug#63645] [PATCH v2] " Brian Cully via Guix-patches via
2023-06-19 10:14   ` bug#63645: " Christopher Baines

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