unofficial mirror of guix-patches@gnu.org 
 help / color / mirror / code / Atom feed
* [bug#69669] [PATCH gnome-team 1/1] gnu: girara: Update to 0.4.3.
  2024-03-09  7:38 [bug#69669] [PATCH gnome-team 0/1] Update girara to 0.4.3 Vivien Kraus via Guix-patches via
@ 2024-03-09  7:33 ` Vivien Kraus via Guix-patches via
  2024-03-09  8:24   ` Liliana Marie Prikler
  2024-03-09  8:58 ` [bug#69669] [PATCH gnome-team v3] gnu: girara: Build with json-glib Vivien Kraus via Guix-patches via
  2024-03-09  9:00 ` [bug#69669] [PATCH gnome-team v2 0/3] Update girara (better) Vivien Kraus via Guix-patches via
  2 siblings, 1 reply; 17+ messages in thread
From: Vivien Kraus via Guix-patches via @ 2024-03-09  7:33 UTC (permalink / raw)
  To: 69669; +Cc: rg, vivien, liliana.prikler, maxim.cournoyer

* gnu/packages/gtk.scm (girara): Update to 0.4.3.
[native-inputs]: Drop labels.
[propagated-inputs]: Add pango and json-glib.
[arguments]: Convert to list of G-Expressions.

Change-Id: I3c977cd33fa6d20fce3bb575ae2ad887b3061966
---
 gnu/packages/gtk.scm | 53 +++++++++++++++++++++++---------------------
 1 file changed, 28 insertions(+), 25 deletions(-)

diff --git a/gnu/packages/gtk.scm b/gnu/packages/gtk.scm
index a00ce71b81..0c1b7fb5a0 100644
--- a/gnu/packages/gtk.scm
+++ b/gnu/packages/gtk.scm
@@ -2132,7 +2132,7 @@ (define-public perl-pango
 (define-public girara
   (package
     (name "girara")
-    (version "0.3.7")
+    (version "0.4.3")
     (source
      (origin
        (method git-fetch)
@@ -2141,31 +2141,34 @@ (define-public girara
              (commit version)))
        (file-name (git-file-name name version))
        (sha256
-        (base32 "0k93pi0lkf941vanvh1habm6n5wl1n63726j5kqxh34wdlv4mv4s"))))
-    (native-inputs `(("pkg-config" ,pkg-config)
-                     ("check" ,check)
-                     ("gettext" ,gettext-minimal)
-                     ("glib:bin" ,glib "bin")
-                     ("xorg-server" ,xorg-server-for-tests)))
-    ;; Listed in 'Requires.private' of 'girara.pc'.
-    (propagated-inputs (list gtk+))
+        (base32 "0cbcs3810frgdmal5ia9pf3rk3k5h4xyzw1d2ia3rcg4nms5gcpx"))))
+    (native-inputs
+     (list pkg-config
+           check
+           gettext-minimal
+           `(,glib "bin")
+           xorg-server-for-tests))
+    ;; Listed in 'Requires.private' or 'Requires' of 'girara.pc'.
+    (propagated-inputs (list gtk+ pango json-glib))
     (arguments
-     `(#:phases (modify-phases %standard-phases
-                  (add-before 'check 'start-xserver
-                    ;; Tests require a running X server.
-                    (lambda* (#:key inputs #:allow-other-keys)
-                      (let ((xorg-server (assoc-ref inputs "xorg-server"))
-                            (display ":1"))
-                        (setenv "DISPLAY" display)
-
-                        ;; On busy machines, tests may take longer than
-                        ;; the default of four seconds.
-                        (setenv "CK_DEFAULT_TIMEOUT" "20")
-
-                        ;; Don't fail due to missing '/etc/machine-id'.
-                        (setenv "DBUS_FATAL_WARNINGS" "0")
-                        (zero? (system (string-append xorg-server "/bin/Xvfb "
-                                                      display " &")))))))))
+     (list
+      #:phases
+      #~(modify-phases %standard-phases
+          (add-before 'check 'start-xserver
+            ;; Tests require a running X server.
+            (lambda* (#:key inputs #:allow-other-keys)
+              (let ((xorg-server (assoc-ref inputs "xorg-server"))
+                    (display ":1"))
+                (setenv "DISPLAY" display)
+
+                ;; On busy machines, tests may take longer than
+                ;; the default of four seconds.
+                (setenv "CK_DEFAULT_TIMEOUT" "20")
+
+                ;; Don't fail due to missing '/etc/machine-id'.
+                (setenv "DBUS_FATAL_WARNINGS" "0")
+                (zero? (system (string-append xorg-server "/bin/Xvfb "
+                                              display " &")))))))))
     (build-system meson-build-system)
     (home-page "https://pwmt.org/projects/girara/")
     (synopsis "Library for minimalistic gtk+3 user interfaces")
-- 
2.41.0




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

* [bug#69669] [PATCH gnome-team v2 2/3] gnu: girara: Update to 0.4.3.
  2024-03-09  9:00 ` [bug#69669] [PATCH gnome-team v2 0/3] Update girara (better) Vivien Kraus via Guix-patches via
@ 2024-03-09  7:33   ` Vivien Kraus via Guix-patches via
  2024-03-09  8:54   ` [bug#69669] [PATCH gnome-team v2 1/3] gnu: girara: Update style Vivien Kraus via Guix-patches via
                     ` (3 subsequent siblings)
  4 siblings, 0 replies; 17+ messages in thread
From: Vivien Kraus via Guix-patches via @ 2024-03-09  7:33 UTC (permalink / raw)
  To: 69669; +Cc: rg, vivien, liliana.prikler, maxim.cournoyer

* gnu/packages/gtk.scm (girara): Update to 0.4.3.

Change-Id: I3c977cd33fa6d20fce3bb575ae2ad887b3061966
---
 gnu/packages/gtk.scm | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/gnu/packages/gtk.scm b/gnu/packages/gtk.scm
index 033c5aee0f..950476815f 100644
--- a/gnu/packages/gtk.scm
+++ b/gnu/packages/gtk.scm
@@ -2132,7 +2132,7 @@ (define-public perl-pango
 (define-public girara
   (package
     (name "girara")
-    (version "0.3.7")
+    (version "0.4.3")
     (source
      (origin
        (method git-fetch)
@@ -2141,7 +2141,7 @@ (define-public girara
              (commit version)))
        (file-name (git-file-name name version))
        (sha256
-        (base32 "0k93pi0lkf941vanvh1habm6n5wl1n63726j5kqxh34wdlv4mv4s"))))
+        (base32 "0cbcs3810frgdmal5ia9pf3rk3k5h4xyzw1d2ia3rcg4nms5gcpx"))))
     (arguments
      (list
       #:phases
-- 
2.41.0




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

* [bug#69669] [PATCH gnome-team 0/1] Update girara to 0.4.3
@ 2024-03-09  7:38 Vivien Kraus via Guix-patches via
  2024-03-09  7:33 ` [bug#69669] [PATCH gnome-team 1/1] gnu: girara: Update " Vivien Kraus via Guix-patches via
                   ` (2 more replies)
  0 siblings, 3 replies; 17+ messages in thread
From: Vivien Kraus via Guix-patches via @ 2024-03-09  7:38 UTC (permalink / raw)
  To: 69669; +Cc: liliana.prikler, maxim.cournoyer, rg, vivien

Dear Guix,

girara fails to build, but we can still update it.  It has an optional
dependency on json-glib, so maybe we could have it too.

What do you think?

Best regards,

Vivien

Vivien Kraus (1):
  gnu: girara: Update to 0.4.3.

 gnu/packages/gtk.scm | 53 +++++++++++++++++++++++---------------------
 1 file changed, 28 insertions(+), 25 deletions(-)


base-commit: 25c14c893f05019d746321285acf55d1aa65b943
-- 
2.41.0




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

* [bug#69669] [PATCH gnome-team 1/1] gnu: girara: Update to 0.4.3.
  2024-03-09  7:33 ` [bug#69669] [PATCH gnome-team 1/1] gnu: girara: Update " Vivien Kraus via Guix-patches via
@ 2024-03-09  8:24   ` Liliana Marie Prikler
  0 siblings, 0 replies; 17+ messages in thread
From: Liliana Marie Prikler @ 2024-03-09  8:24 UTC (permalink / raw)
  To: Vivien Kraus, 69669; +Cc: rg, maxim.cournoyer

Am Samstag, dem 09.03.2024 um 08:33 +0100 schrieb Vivien Kraus:
> * gnu/packages/gtk.scm (girara): Update to 0.4.3.
> [native-inputs]: Drop labels.
> [propagated-inputs]: Add pango and json-glib.
> [arguments]: Convert to list of G-Expressions.
> 
> Change-Id: I3c977cd33fa6d20fce3bb575ae2ad887b3061966
> ---
Since you are already mixing cosmetic and non-cosmetic changes, you can
also sort the fields.  Inputs typically go after arguments.
>  gnu/packages/gtk.scm | 53 +++++++++++++++++++++++-------------------
> --
>  1 file changed, 28 insertions(+), 25 deletions(-)
> 
> diff --git a/gnu/packages/gtk.scm b/gnu/packages/gtk.scm
> index a00ce71b81..0c1b7fb5a0 100644
> --- a/gnu/packages/gtk.scm
> +++ b/gnu/packages/gtk.scm
> @@ -2132,7 +2132,7 @@ (define-public perl-pango
>  (define-public girara
>    (package
>      (name "girara")
> -    (version "0.3.7")
> +    (version "0.4.3")
>      (source
>       (origin
>         (method git-fetch)
> @@ -2141,31 +2141,34 @@ (define-public girara
>               (commit version)))
>         (file-name (git-file-name name version))
>         (sha256
> -        (base32
> "0k93pi0lkf941vanvh1habm6n5wl1n63726j5kqxh34wdlv4mv4s"))))
> -    (native-inputs `(("pkg-config" ,pkg-config)
> -                     ("check" ,check)
> -                     ("gettext" ,gettext-minimal)
> -                     ("glib:bin" ,glib "bin")
> -                     ("xorg-server" ,xorg-server-for-tests)))
> -    ;; Listed in 'Requires.private' of 'girara.pc'.
> -    (propagated-inputs (list gtk+))
> +        (base32
> "0cbcs3810frgdmal5ia9pf3rk3k5h4xyzw1d2ia3rcg4nms5gcpx"))))
> +    (native-inputs
> +     (list pkg-config
> +           check
> +           gettext-minimal
> +           `(,glib "bin")
> +           xorg-server-for-tests))
> +    ;; Listed in 'Requires.private' or 'Requires' of 'girara.pc'.
> +    (propagated-inputs (list gtk+ pango json-glib))
I'd hazard a guess that some of these are already propagated through
gtk+.
>      (arguments
> -     `(#:phases (modify-phases %standard-phases
> -                  (add-before 'check 'start-xserver
> -                    ;; Tests require a running X server.
> -                    (lambda* (#:key inputs #:allow-other-keys)
> -                      (let ((xorg-server (assoc-ref inputs "xorg-
> server"))
> -                            (display ":1"))
> -                        (setenv "DISPLAY" display)
> -
> -                        ;; On busy machines, tests may take longer
> than
> -                        ;; the default of four seconds.
> -                        (setenv "CK_DEFAULT_TIMEOUT" "20")
> -
> -                        ;; Don't fail due to missing '/etc/machine-
> id'.
> -                        (setenv "DBUS_FATAL_WARNINGS" "0")
> -                        (zero? (system (string-append xorg-server
> "/bin/Xvfb "
> -                                                      display "
> &")))))))))
> +     (list
> +      #:phases
> +      #~(modify-phases %standard-phases
> +          (add-before 'check 'start-xserver
> +            ;; Tests require a running X server.
> +            (lambda* (#:key inputs #:allow-other-keys)
> +              (let ((xorg-server (assoc-ref inputs "xorg-server"))
> +                    (display ":1"))
> +                (setenv "DISPLAY" display)
> +
> +                ;; On busy machines, tests may take longer than
> +                ;; the default of four seconds.
> +                (setenv "CK_DEFAULT_TIMEOUT" "20")
> +
> +                ;; Don't fail due to missing '/etc/machine-id'.
> +                (setenv "DBUS_FATAL_WARNINGS" "0")
> +                (zero? (system (string-append xorg-server "/bin/Xvfb
> "
> +                                              display " &")))))))))
>      (build-system meson-build-system)
>      (home-page "https://pwmt.org/projects/girara/")
>      (synopsis "Library for minimalistic gtk+3 user interfaces")
Cheers

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

* [bug#69669] [PATCH gnome-team v2 1/3] gnu: girara: Update style.
  2024-03-09  9:00 ` [bug#69669] [PATCH gnome-team v2 0/3] Update girara (better) Vivien Kraus via Guix-patches via
  2024-03-09  7:33   ` [bug#69669] [PATCH gnome-team v2 2/3] gnu: girara: Update to 0.4.3 Vivien Kraus via Guix-patches via
@ 2024-03-09  8:54   ` Vivien Kraus via Guix-patches via
  2024-03-09  8:58   ` [bug#69669] [PATCH gnome-team v2 3/3] gnu: girara: Build with json-glib Vivien Kraus via Guix-patches via
                     ` (2 subsequent siblings)
  4 siblings, 0 replies; 17+ messages in thread
From: Vivien Kraus via Guix-patches via @ 2024-03-09  8:54 UTC (permalink / raw)
  To: 69669; +Cc: rg, vivien, liliana.prikler, maxim.cournoyer

* gnu/packages/gtk.scm (girara)[arguments]: Convert to list of G-Expressions.
[native-inputs]: Move after arguments.
[propagated-inputs]: Move after arguments.

Change-Id: I79a6f59f6ea671433f1a66a10cb79203f0b9bbae
---
 gnu/packages/gtk.scm | 47 +++++++++++++++++++++++---------------------
 1 file changed, 25 insertions(+), 22 deletions(-)

diff --git a/gnu/packages/gtk.scm b/gnu/packages/gtk.scm
index a00ce71b81..033c5aee0f 100644
--- a/gnu/packages/gtk.scm
+++ b/gnu/packages/gtk.scm
@@ -2142,30 +2142,33 @@ (define-public girara
        (file-name (git-file-name name version))
        (sha256
         (base32 "0k93pi0lkf941vanvh1habm6n5wl1n63726j5kqxh34wdlv4mv4s"))))
-    (native-inputs `(("pkg-config" ,pkg-config)
-                     ("check" ,check)
-                     ("gettext" ,gettext-minimal)
-                     ("glib:bin" ,glib "bin")
-                     ("xorg-server" ,xorg-server-for-tests)))
+    (arguments
+     (list
+      #:phases
+      #~(modify-phases %standard-phases
+          (add-before 'check 'start-xserver
+            ;; Tests require a running X server.
+            (lambda* (#:key inputs #:allow-other-keys)
+              (let ((xorg-server (assoc-ref inputs "xorg-server"))
+                    (display ":1"))
+                (setenv "DISPLAY" display)
+
+                ;; On busy machines, tests may take longer than
+                ;; the default of four seconds.
+                (setenv "CK_DEFAULT_TIMEOUT" "20")
+
+                ;; Don't fail due to missing '/etc/machine-id'.
+                (setenv "DBUS_FATAL_WARNINGS" "0")
+                (zero? (system (string-append xorg-server "/bin/Xvfb "
+                                              display " &")))))))))
+    (native-inputs
+     (list pkg-config
+           check
+           gettext-minimal
+           `(,glib "bin")
+           xorg-server-for-tests))
     ;; Listed in 'Requires.private' of 'girara.pc'.
     (propagated-inputs (list gtk+))
-    (arguments
-     `(#:phases (modify-phases %standard-phases
-                  (add-before 'check 'start-xserver
-                    ;; Tests require a running X server.
-                    (lambda* (#:key inputs #:allow-other-keys)
-                      (let ((xorg-server (assoc-ref inputs "xorg-server"))
-                            (display ":1"))
-                        (setenv "DISPLAY" display)
-
-                        ;; On busy machines, tests may take longer than
-                        ;; the default of four seconds.
-                        (setenv "CK_DEFAULT_TIMEOUT" "20")
-
-                        ;; Don't fail due to missing '/etc/machine-id'.
-                        (setenv "DBUS_FATAL_WARNINGS" "0")
-                        (zero? (system (string-append xorg-server "/bin/Xvfb "
-                                                      display " &")))))))))
     (build-system meson-build-system)
     (home-page "https://pwmt.org/projects/girara/")
     (synopsis "Library for minimalistic gtk+3 user interfaces")
-- 
2.41.0




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

* [bug#69669] [PATCH gnome-team v2 3/3] gnu: girara: Build with json-glib.
  2024-03-09  9:00 ` [bug#69669] [PATCH gnome-team v2 0/3] Update girara (better) Vivien Kraus via Guix-patches via
  2024-03-09  7:33   ` [bug#69669] [PATCH gnome-team v2 2/3] gnu: girara: Update to 0.4.3 Vivien Kraus via Guix-patches via
  2024-03-09  8:54   ` [bug#69669] [PATCH gnome-team v2 1/3] gnu: girara: Update style Vivien Kraus via Guix-patches via
@ 2024-03-09  8:58   ` Vivien Kraus via Guix-patches via
  2024-03-09 17:55     ` Maxim Cournoyer
  2024-03-09  9:24   ` [bug#69669] [PATCH gnome-team v2 0/3] Update girara (better) Liliana Marie Prikler
  2024-03-09 17:58   ` Maxim Cournoyer
  4 siblings, 1 reply; 17+ messages in thread
From: Vivien Kraus via Guix-patches via @ 2024-03-09  8:58 UTC (permalink / raw)
  To: 69669; +Cc: rg, vivien, liliana.prikler, maxim.cournoyer

* gnu/packages/gtk.scm (girara) [propagated-inputs]: Add json-glib.

Change-Id: I5bfe5a516885153014fce1c6643ac5799f0100e1
---
 gnu/packages/gtk.scm | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/gnu/packages/gtk.scm b/gnu/packages/gtk.scm
index 950476815f..3b834d748c 100644
--- a/gnu/packages/gtk.scm
+++ b/gnu/packages/gtk.scm
@@ -2168,7 +2168,7 @@ (define-public girara
            `(,glib "bin")
            xorg-server-for-tests))
     ;; Listed in 'Requires.private' of 'girara.pc'.
-    (propagated-inputs (list gtk+))
+    (propagated-inputs (list gtk+ json-glib))
     (build-system meson-build-system)
     (home-page "https://pwmt.org/projects/girara/")
     (synopsis "Library for minimalistic gtk+3 user interfaces")
-- 
2.41.0




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

* [bug#69669] [PATCH gnome-team v3] gnu: girara: Build with json-glib.
  2024-03-09  7:38 [bug#69669] [PATCH gnome-team 0/1] Update girara to 0.4.3 Vivien Kraus via Guix-patches via
  2024-03-09  7:33 ` [bug#69669] [PATCH gnome-team 1/1] gnu: girara: Update " Vivien Kraus via Guix-patches via
@ 2024-03-09  8:58 ` Vivien Kraus via Guix-patches via
  2024-03-12 13:28   ` Maxim Cournoyer
  2024-03-09  9:00 ` [bug#69669] [PATCH gnome-team v2 0/3] Update girara (better) Vivien Kraus via Guix-patches via
  2 siblings, 1 reply; 17+ messages in thread
From: Vivien Kraus via Guix-patches via @ 2024-03-09  8:58 UTC (permalink / raw)
  To: 69669; +Cc: rg, vivien, liliana.prikler, maxim.cournoyer

* gnu/packages/gtk.scm (girara) [propagated-inputs]: Add json-glib.

Change-Id: I5bfe5a516885153014fce1c6643ac5799f0100e1
---
The added comment has a date, so that the maintainer of the future knows
whether they can safely apply the change.

Best regards,

Vivien

 gnu/packages/gtk.scm | 4 +++-
 1 file changed, 3 insertions(+), 1 deletion(-)

diff --git a/gnu/packages/gtk.scm b/gnu/packages/gtk.scm
index 950476815f..abbfc6fa16 100644
--- a/gnu/packages/gtk.scm
+++ b/gnu/packages/gtk.scm
@@ -2130,6 +2130,8 @@ (define-public perl-pango
     (license license:lgpl2.1+)))
 
 (define-public girara
+  ;; TODO: Move propagated inputs to inputs after core-updates is merged (as
+  ;; of 2024-03)
   (package
     (name "girara")
     (version "0.4.3")
@@ -2168,7 +2170,7 @@ (define-public girara
            `(,glib "bin")
            xorg-server-for-tests))
     ;; Listed in 'Requires.private' of 'girara.pc'.
-    (propagated-inputs (list gtk+))
+    (propagated-inputs (list gtk+ json-glib))
     (build-system meson-build-system)
     (home-page "https://pwmt.org/projects/girara/")
     (synopsis "Library for minimalistic gtk+3 user interfaces")

base-commit: fe2b2f860e1fd7dfdc333f65893e65f131e290c7
-- 
2.41.0




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

* [bug#69669] [PATCH gnome-team v2 0/3] Update girara (better)
  2024-03-09  7:38 [bug#69669] [PATCH gnome-team 0/1] Update girara to 0.4.3 Vivien Kraus via Guix-patches via
  2024-03-09  7:33 ` [bug#69669] [PATCH gnome-team 1/1] gnu: girara: Update " Vivien Kraus via Guix-patches via
  2024-03-09  8:58 ` [bug#69669] [PATCH gnome-team v3] gnu: girara: Build with json-glib Vivien Kraus via Guix-patches via
@ 2024-03-09  9:00 ` Vivien Kraus via Guix-patches via
  2024-03-09  7:33   ` [bug#69669] [PATCH gnome-team v2 2/3] gnu: girara: Update to 0.4.3 Vivien Kraus via Guix-patches via
                     ` (4 more replies)
  2 siblings, 5 replies; 17+ messages in thread
From: Vivien Kraus via Guix-patches via @ 2024-03-09  9:00 UTC (permalink / raw)
  To: 69669; +Cc: rg, vivien, liliana.prikler, maxim.cournoyer

Dear guix,

I split the cosmetic changes, and also split the non-cosmetic change in 2.
json-glib is an optional dependency, so girara can exist with and without it.

You were right, pango is already propagated from gtk+.

I also reordered the fields, but it does not look very intuitive, because it
looks like I moved the arguments field up and the native-inputs field down
instead of moving native-inputs and propagated-inputs down (thanks git).
Should I also split the style update?

Best regards,

Vivien

Vivien Kraus (3):
  gnu: girara: Update style.
  gnu: girara: Update to 0.4.3.
  gnu: girara: Build with json-glib.

 gnu/packages/gtk.scm | 53 +++++++++++++++++++++++---------------------
 1 file changed, 28 insertions(+), 25 deletions(-)


base-commit: 25c14c893f05019d746321285acf55d1aa65b943
-- 
2.41.0




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

* [bug#69669] [PATCH gnome-team v2 0/3] Update girara (better)
  2024-03-09  9:00 ` [bug#69669] [PATCH gnome-team v2 0/3] Update girara (better) Vivien Kraus via Guix-patches via
                     ` (2 preceding siblings ...)
  2024-03-09  8:58   ` [bug#69669] [PATCH gnome-team v2 3/3] gnu: girara: Build with json-glib Vivien Kraus via Guix-patches via
@ 2024-03-09  9:24   ` Liliana Marie Prikler
  2024-03-09 12:33     ` Vivien Kraus via Guix-patches via
  2024-03-09 17:58   ` Maxim Cournoyer
  4 siblings, 1 reply; 17+ messages in thread
From: Liliana Marie Prikler @ 2024-03-09  9:24 UTC (permalink / raw)
  To: Vivien Kraus, 69669; +Cc: rg, maxim.cournoyer

Am Samstag, dem 09.03.2024 um 10:00 +0100 schrieb Vivien Kraus:
> Dear guix,
> 
> I split the cosmetic changes, and also split the non-cosmetic change
> in 2.  json-glib is an optional dependency, so girara can exist with
> and without it.
Would you care to elaborate?  Depending on the exact changes between
the version, I might want to squash this into one commit.

> You were right, pango is already propagated from gtk+.
> 
> I also reordered the fields, but it does not look very intuitive,
> because it looks like I moved the arguments field up and the native-
> inputs field down instead of moving native-inputs and propagated-
> inputs down (thanks git).
> Should I also split the style update?
Nah, that's fine.

Cheers




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

* [bug#69669] [PATCH gnome-team v2 0/3] Update girara (better)
  2024-03-09  9:24   ` [bug#69669] [PATCH gnome-team v2 0/3] Update girara (better) Liliana Marie Prikler
@ 2024-03-09 12:33     ` Vivien Kraus via Guix-patches via
  0 siblings, 0 replies; 17+ messages in thread
From: Vivien Kraus via Guix-patches via @ 2024-03-09 12:33 UTC (permalink / raw)
  To: Liliana Marie Prikler, 69669; +Cc: rg, maxim.cournoyer

Le samedi 09 mars 2024 à 10:24 +0100, Liliana Marie Prikler a écrit :
> > I split the cosmetic changes, and also split the non-cosmetic
> > change
> > in 2.  json-glib is an optional dependency, so girara can exist
> > with
> > and without it.
> Would you care to elaborate?  Depending on the exact changes between
> the version, I might want to squash this into one commit.

In meson_options.txt, here is the description for the 'json' option: 

```
description: 'dump configuration using JSON'
```

The README has:

```
The following dependencies are optional:

* `json-glib-1.0`: configuration dumping support
```

The blame tells us it was added in september 2022, so later than the
0.3.7 release (february 2022).

I don’t really understand what it’s used for, I have never developed
with girara.

Does this help?

Vivien




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

* [bug#69669] [PATCH gnome-team v2 3/3] gnu: girara: Build with json-glib.
  2024-03-09  8:58   ` [bug#69669] [PATCH gnome-team v2 3/3] gnu: girara: Build with json-glib Vivien Kraus via Guix-patches via
@ 2024-03-09 17:55     ` Maxim Cournoyer
  2024-03-09 18:22       ` Vivien Kraus via Guix-patches via
  0 siblings, 1 reply; 17+ messages in thread
From: Maxim Cournoyer @ 2024-03-09 17:55 UTC (permalink / raw)
  To: Vivien Kraus; +Cc: rg, liliana.prikler, 69669

Hi Vivien,

Vivien Kraus <vivien@planete-kraus.eu> writes:

> * gnu/packages/gtk.scm (girara) [propagated-inputs]: Add json-glib.
>
> Change-Id: I5bfe5a516885153014fce1c6643ac5799f0100e1
> ---
>  gnu/packages/gtk.scm | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/gnu/packages/gtk.scm b/gnu/packages/gtk.scm
> index 950476815f..3b834d748c 100644
> --- a/gnu/packages/gtk.scm
> +++ b/gnu/packages/gtk.scm
> @@ -2168,7 +2168,7 @@ (define-public girara
>             `(,glib "bin")
>             xorg-server-for-tests))
>      ;; Listed in 'Requires.private' of 'girara.pc'.
> -    (propagated-inputs (list gtk+))
> +    (propagated-inputs (list gtk+ json-glib))

We'll want to revisit these propagated inputs after the core-updates
branch is merged, as Requires.private should only be needed when
producing static archives, not dynamic libraries (which pkgconf
interprets correctly).

-- 
Thanks,
Maxim




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

* [bug#69669] [PATCH gnome-team v2 0/3] Update girara (better)
  2024-03-09  9:00 ` [bug#69669] [PATCH gnome-team v2 0/3] Update girara (better) Vivien Kraus via Guix-patches via
                     ` (3 preceding siblings ...)
  2024-03-09  9:24   ` [bug#69669] [PATCH gnome-team v2 0/3] Update girara (better) Liliana Marie Prikler
@ 2024-03-09 17:58   ` Maxim Cournoyer
  2024-03-09 18:40     ` Liliana Marie Prikler
  4 siblings, 1 reply; 17+ messages in thread
From: Maxim Cournoyer @ 2024-03-09 17:58 UTC (permalink / raw)
  To: Vivien Kraus; +Cc: rg, liliana.prikler, 69669

Hello,

Vivien Kraus <vivien@planete-kraus.eu> writes:

> Dear guix,
>
> I split the cosmetic changes, and also split the non-cosmetic change in 2.
> json-glib is an optional dependency, so girara can exist with and without it.
>
> You were right, pango is already propagated from gtk+.
>
> I also reordered the fields, but it does not look very intuitive, because it
> looks like I moved the arguments field up and the native-inputs field down
> instead of moving native-inputs and propagated-inputs down (thanks git).
> Should I also split the style update?
>
> Best regards,
>
> Vivien
>
> Vivien Kraus (3):
>   gnu: girara: Update style.
>   gnu: girara: Update to 0.4.3.
>   gnu: girara: Build with json-glib.
>

Reviewed-by: Maxim Cournoyer <maxim.cournoyer@gmail>

-- 
Thanks,
Maxim




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

* [bug#69669] [PATCH gnome-team v2 3/3] gnu: girara: Build with json-glib.
  2024-03-09 17:55     ` Maxim Cournoyer
@ 2024-03-09 18:22       ` Vivien Kraus via Guix-patches via
  2024-03-12  3:47         ` Maxim Cournoyer
  0 siblings, 1 reply; 17+ messages in thread
From: Vivien Kraus via Guix-patches via @ 2024-03-09 18:22 UTC (permalink / raw)
  To: Maxim Cournoyer; +Cc: rg, liliana.prikler, 69669

Le samedi 09 mars 2024 à 12:55 -0500, Maxim Cournoyer a écrit :
> > -    (propagated-inputs (list gtk+))
> > +    (propagated-inputs (list gtk+ json-glib))
> 
> We'll want to revisit these propagated inputs after the core-updates
> branch is merged, as Requires.private should only be needed when
> producing static archives, not dynamic libraries (which pkgconf
> interprets correctly).

Should I do something in the mean time?




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

* [bug#69669] [PATCH gnome-team v2 0/3] Update girara (better)
  2024-03-09 17:58   ` Maxim Cournoyer
@ 2024-03-09 18:40     ` Liliana Marie Prikler
  0 siblings, 0 replies; 17+ messages in thread
From: Liliana Marie Prikler @ 2024-03-09 18:40 UTC (permalink / raw)
  To: Maxim Cournoyer, Vivien Kraus; +Cc: rg, 69669

Am Samstag, dem 09.03.2024 um 12:58 -0500 schrieb Maxim Cournoyer:
> Hello,
> 
> Vivien Kraus <vivien@planete-kraus.eu> writes:
> 
> > Dear guix,
> > 
> > I split the cosmetic changes, and also split the non-cosmetic
> > change in 2.
> > json-glib is an optional dependency, so girara can exist with and
> > without it.
> > 
> > You were right, pango is already propagated from gtk+.
> > 
> > I also reordered the fields, but it does not look very intuitive,
> > because it looks like I moved the arguments field up and the
> > native-inputs field down instead of moving native-inputs and
> > propagated-inputs down (thanks git).
> > Should I also split the style update?
> > 
> > Best regards,
> > 
> > Vivien
> > 
> > Vivien Kraus (3):
> >   gnu: girara: Update style.
> >   gnu: girara: Update to 0.4.3.
> >   gnu: girara: Build with json-glib.
> > 
> 
> Reviewed-by: Maxim Cournoyer <maxim.cournoyer@gmail>
Hi, I've went ahead and pushed 1 and 2.  For 3, I'd like you to clarify
whether we ought to use propagated-inputs now or how else to handle
this.

Cheers





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

* [bug#69669] [PATCH gnome-team v2 3/3] gnu: girara: Build with json-glib.
  2024-03-09 18:22       ` Vivien Kraus via Guix-patches via
@ 2024-03-12  3:47         ` Maxim Cournoyer
  0 siblings, 0 replies; 17+ messages in thread
From: Maxim Cournoyer @ 2024-03-12  3:47 UTC (permalink / raw)
  To: Vivien Kraus; +Cc: rg, liliana.prikler, 69669

Hi,

Vivien Kraus <vivien@planete-kraus.eu> writes:

> Le samedi 09 mars 2024 à 12:55 -0500, Maxim Cournoyer a écrit :
>> > -    (propagated-inputs (list gtk+))
>> > +    (propagated-inputs (list gtk+ json-glib))
>> 
>> We'll want to revisit these propagated inputs after the core-updates
>> branch is merged, as Requires.private should only be needed when
>> producing static archives, not dynamic libraries (which pkgconf
>> interprets correctly).
>
> Should I do something in the mean time?

No, or if you want, you could leave a 'TODO: Remove propagated inputs
after core-updates is merged' or similar.

-- 
Thanks,
Maxim




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

* [bug#69669] [PATCH gnome-team v3] gnu: girara: Build with json-glib.
  2024-03-09  8:58 ` [bug#69669] [PATCH gnome-team v3] gnu: girara: Build with json-glib Vivien Kraus via Guix-patches via
@ 2024-03-12 13:28   ` Maxim Cournoyer
  2024-03-16 11:05     ` Christopher Baines
  0 siblings, 1 reply; 17+ messages in thread
From: Maxim Cournoyer @ 2024-03-12 13:28 UTC (permalink / raw)
  To: Vivien Kraus; +Cc: rg, liliana.prikler, 69669

Vivien Kraus <vivien@planete-kraus.eu> writes:

> * gnu/packages/gtk.scm (girara) [propagated-inputs]: Add json-glib.
>
> Change-Id: I5bfe5a516885153014fce1c6643ac5799f0100e1

Reviewed-by: Maxim Cournoyer <maxim.cournoyer@gmail>

-- 
Thanks,
Maxim




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

* [bug#69669] [PATCH gnome-team v3] gnu: girara: Build with json-glib.
  2024-03-12 13:28   ` Maxim Cournoyer
@ 2024-03-16 11:05     ` Christopher Baines
  0 siblings, 0 replies; 17+ messages in thread
From: Christopher Baines @ 2024-03-16 11:05 UTC (permalink / raw)
  To: Maxim Cournoyer; +Cc: rg, vivien, liliana.prikler, 69669, 69669-done

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


Maxim Cournoyer <maxim.cournoyer@gmail.com> writes:

> Vivien Kraus <vivien@planete-kraus.eu> writes:
>
>> * gnu/packages/gtk.scm (girara) [propagated-inputs]: Add json-glib.
>>
>> Change-Id: I5bfe5a516885153014fce1c6643ac5799f0100e1
>
> Reviewed-by: Maxim Cournoyer <maxim.cournoyer@gmail>

Thanks all, I've pushed this to gnome-team as
1caa0eeee5042f33bc0bb66dc38bed7513cad0fb.

Chris

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

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

end of thread, other threads:[~2024-03-16 11:07 UTC | newest]

Thread overview: 17+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2024-03-09  7:38 [bug#69669] [PATCH gnome-team 0/1] Update girara to 0.4.3 Vivien Kraus via Guix-patches via
2024-03-09  7:33 ` [bug#69669] [PATCH gnome-team 1/1] gnu: girara: Update " Vivien Kraus via Guix-patches via
2024-03-09  8:24   ` Liliana Marie Prikler
2024-03-09  8:58 ` [bug#69669] [PATCH gnome-team v3] gnu: girara: Build with json-glib Vivien Kraus via Guix-patches via
2024-03-12 13:28   ` Maxim Cournoyer
2024-03-16 11:05     ` Christopher Baines
2024-03-09  9:00 ` [bug#69669] [PATCH gnome-team v2 0/3] Update girara (better) Vivien Kraus via Guix-patches via
2024-03-09  7:33   ` [bug#69669] [PATCH gnome-team v2 2/3] gnu: girara: Update to 0.4.3 Vivien Kraus via Guix-patches via
2024-03-09  8:54   ` [bug#69669] [PATCH gnome-team v2 1/3] gnu: girara: Update style Vivien Kraus via Guix-patches via
2024-03-09  8:58   ` [bug#69669] [PATCH gnome-team v2 3/3] gnu: girara: Build with json-glib Vivien Kraus via Guix-patches via
2024-03-09 17:55     ` Maxim Cournoyer
2024-03-09 18:22       ` Vivien Kraus via Guix-patches via
2024-03-12  3:47         ` Maxim Cournoyer
2024-03-09  9:24   ` [bug#69669] [PATCH gnome-team v2 0/3] Update girara (better) Liliana Marie Prikler
2024-03-09 12:33     ` Vivien Kraus via Guix-patches via
2024-03-09 17:58   ` Maxim Cournoyer
2024-03-09 18:40     ` Liliana Marie Prikler

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