unofficial mirror of guix-patches@gnu.org 
 help / color / mirror / code / Atom feed
* [bug#37693] Seahorse build
@ 2019-10-10  8:56 Leo Prikler
  2019-10-13 12:35 ` Leo Prikler
                   ` (3 more replies)
  0 siblings, 4 replies; 9+ messages in thread
From: Leo Prikler @ 2019-10-10  8:56 UTC (permalink / raw)
  To: 37693

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



[-- Attachment #2: 0001-gnu-seahorse-Fix-build.patch --]
[-- Type: text/x-patch, Size: 9352 bytes --]

From e2b17f74841ae14ac96782539ec8bfc1c639ed34 Mon Sep 17 00:00:00 2001
From: Leo Prikler <leo.prikler@student.tugraz.at>
Date: Thu, 10 Oct 2019 10:16:47 +0200
Subject: [PATCH] gnu: seahorse: Fix build

* /gnu/packages/gnome.scm: (seahorse): Use meson-build-system.
Update inputs. Add patches to fix compile errors.
* /gnu/packages/patches/seahorse-fix-libsecret-breakage.patch: New file.
* /gnu/packages/patches/seahorse-fix-type-mismatches.patch: New file.
---
 gnu/packages/gnome.scm                        |  12 +-
 .../seahorse-fix-libsecret-breakage.patch     |  29 ++++
 .../seahorse-fix-type-mismatches.patch        | 154 ++++++++++++++++++
 3 files changed, 192 insertions(+), 3 deletions(-)
 create mode 100644 gnu/packages/patches/seahorse-fix-libsecret-breakage.patch
 create mode 100644 gnu/packages/patches/seahorse-fix-type-mismatches.patch

diff --git a/gnu/packages/gnome.scm b/gnu/packages/gnome.scm
index 2a664e3db7..a34babebfe 100644
--- a/gnu/packages/gnome.scm
+++ b/gnu/packages/gnome.scm
@@ -2230,10 +2230,14 @@ engineering.")
                            version ".tar.xz"))
        (sha256
         (base32
-         "1sbj1czlx1fakm72dwgbn0bwm12j838yaky4mkf6hf8j8afnxmzp"))))
-    (build-system glib-or-gtk-build-system)
+         "1sbj1czlx1fakm72dwgbn0bwm12j838yaky4mkf6hf8j8afnxmzp"))
+       (patches
+        (search-patches "seahorse-fix-libsecret-breakage.patch"
+                        "seahorse-fix-type-mismatches.patch"))))
+    (build-system meson-build-system)
     (inputs
-     `(("gtk+" ,gtk+)
+     `(("avahi" ,avahi)
+       ("gtk+" ,gtk+)
        ("gcr" ,gcr)
        ("gnupg" ,gnupg)
        ("gpgme" ,gpgme)
@@ -2244,8 +2248,10 @@ engineering.")
     (native-inputs
      `(("intltool" ,intltool)
        ("glib:bin" ,glib "bin")
+       ("gtk+:bin" ,gtk+ "bin")
        ("itstool" ,itstool)
        ("pkg-config" ,pkg-config)
+       ("vala" ,vala)
        ("xmllint" ,libxml2)))
     (home-page "https://launchpad.net/gnome-themes-standard")
     (synopsis "Manage encryption keys and passwords in the GNOME keyring")
diff --git a/gnu/packages/patches/seahorse-fix-libsecret-breakage.patch b/gnu/packages/patches/seahorse-fix-libsecret-breakage.patch
new file mode 100644
index 0000000000..d6396c227a
--- /dev/null
+++ b/gnu/packages/patches/seahorse-fix-libsecret-breakage.patch
@@ -0,0 +1,29 @@
+From d9db29db567012b7c72e85e1be1fbf55fcc9b667 Mon Sep 17 00:00:00 2001
+From: Niels De Graef <nielsdegraef@gmail.com>
+Date: Sat, 11 May 2019 09:02:34 +0200
+Subject: [PATCH] gkr: Use 0 on empty flags
+
+A Flags-type variable without any flag set can be replaced with 0, so
+this is a safe thing to do. It also prevents us from having to deal with
+the accidental API break in libsecret (see
+https://gitlab.gnome.org/GNOME/libsecret/merge_requests/19)
+---
+ gkr/gkr-keyring-add.vala | 3 +--
+ 1 file changed, 1 insertion(+), 2 deletions(-)
+
+diff --git a/gkr/gkr-keyring-add.vala b/gkr/gkr-keyring-add.vala
+index 4e92a520..f60c9a22 100644
+--- a/gkr/gkr-keyring-add.vala
++++ b/gkr/gkr-keyring-add.vala
+@@ -43,8 +43,7 @@ public class Seahorse.Gkr.KeyringAdd : Gtk.Dialog {
+
+         var cancellable = Dialog.begin_request(this);
+         var service = Backend.instance().service;
+-        Secret.Collection.create.begin(service, this.name_entry.text, null,
+-                                       Secret.CollectionCreateFlags.COLLECTION_CREATE_NONE,
++        Secret.Collection.create.begin(service, this.name_entry.text, null, 0,
+                                        cancellable, (obj, res) => {
+             /* Clear the operation without cancelling it since it is complete */
+             Dialog.complete_request(this, false);
+--
+2.22.0
diff --git a/gnu/packages/patches/seahorse-fix-type-mismatches.patch b/gnu/packages/patches/seahorse-fix-type-mismatches.patch
new file mode 100644
index 0000000000..d259d8311d
--- /dev/null
+++ b/gnu/packages/patches/seahorse-fix-type-mismatches.patch
@@ -0,0 +1,154 @@
+From 12178d7cc7fad52f61003b3221bd0a713ef80507 Mon Sep 17 00:00:00 2001
+From: Rico Tzschichholz <ricotz@ubuntu.com>
+Date: Mon, 5 Nov 2018 09:59:27 +0100
+Subject: [PATCH] Fix type-argument mismatches related to GLib.List
+
+---
+ common/deleter.vala                  | 2 +-
+ common/exporter.vala                 | 2 +-
+ gkr/gkr-item.vala                    | 2 +-
+ gkr/gkr-keyring.vala                 | 2 +-
+ pkcs11/certificate-der-exporter.vala | 4 ++--
+ pkcs11/pkcs11-deleter.vala           | 2 +-
+ pkcs11/pkcs11-token.vala             | 4 ++--
+ ssh/deleter.vala                     | 2 +-
+ ssh/exporter.vala                    | 2 +-
+ 9 files changed, 11 insertions(+), 11 deletions(-)
+
+diff --git a/common/deleter.vala b/common/deleter.vala
+index 18dd0d4c..1c50cdaf 100644
+--- a/common/deleter.vala
++++ b/common/deleter.vala
+@@ -24,7 +24,7 @@ namespace Seahorse {
+ public abstract class Deleter : GLib.Object {
+ 	public abstract Gtk.Dialog create_confirm(Gtk.Window? parent);
+
+-	public abstract unowned GLib.List<weak GLib.Object> get_objects();
++	public abstract unowned GLib.List<GLib.Object> get_objects();
+
+ 	public abstract bool add_object (GLib.Object obj);
+
+diff --git a/common/exporter.vala b/common/exporter.vala
+index e686ff0d..edc4b4d3 100644
+--- a/common/exporter.vala
++++ b/common/exporter.vala
+@@ -35,7 +35,7 @@ public interface Exporter : GLib.Object {
+
+ 	public abstract Gtk.FileFilter file_filter { owned get; }
+
+-	public abstract unowned GLib.List<weak GLib.Object> get_objects();
++	public abstract unowned GLib.List<GLib.Object> get_objects();
+
+ 	public abstract bool add_object(GLib.Object obj);
+
+diff --git a/gkr/gkr-item.vala b/gkr/gkr-item.vala
+index ec45a5c7..bdcf14d1 100644
+--- a/gkr/gkr-item.vala
++++ b/gkr/gkr-item.vala
+@@ -513,7 +513,7 @@ class ItemDeleter : Deleter {
+ 			GLib.assert_not_reached();
+ 	}
+
+-	public override unowned GLib.List<weak GLib.Object> get_objects() {
++	public override unowned GLib.List<GLib.Object> get_objects() {
+ 		return this._items;
+ 	}
+
+diff --git a/gkr/gkr-keyring.vala b/gkr/gkr-keyring.vala
+index e6ecaf76..096bb0ae 100644
+--- a/gkr/gkr-keyring.vala
++++ b/gkr/gkr-keyring.vala
+@@ -248,7 +248,7 @@ class KeyringDeleter : Deleter {
+ 			GLib.assert_not_reached();
+ 	}
+
+-	public override unowned GLib.List<weak GLib.Object> get_objects() {
++	public override unowned GLib.List<GLib.Object> get_objects() {
+ 		return this._objects;
+ 	}
+
+diff --git a/pkcs11/certificate-der-exporter.vala b/pkcs11/certificate-der-exporter.vala
+index e8789f52..57e6ec25 100644
+--- a/pkcs11/certificate-der-exporter.vala
++++ b/pkcs11/certificate-der-exporter.vala
+@@ -23,7 +23,7 @@ namespace Seahorse {
+
+ public class CertificateDerExporter : GLib.Object, Exporter {
+ 	private Gcr.Certificate _certificate;
+-	private GLib.List<weak GLib.Object> _objects;
++	private GLib.List<GLib.Object> _objects;
+
+ 	public CertificateDerExporter(Gcr.Certificate certificate)
+ 	{
+@@ -67,7 +67,7 @@ public class CertificateDerExporter : GLib.Object, Exporter {
+ 		}
+ 	}
+
+-	public unowned GLib.List<weak GLib.Object> get_objects() {
++	public unowned GLib.List<GLib.Object> get_objects() {
+ 		return this._objects;
+ 	}
+
+diff --git a/pkcs11/pkcs11-deleter.vala b/pkcs11/pkcs11-deleter.vala
+index 34efc0eb..475b30c4 100644
+--- a/pkcs11/pkcs11-deleter.vala
++++ b/pkcs11/pkcs11-deleter.vala
+@@ -44,7 +44,7 @@ public class Deleter : Seahorse.Deleter {
+ 			GLib.assert_not_reached();
+ 	}
+
+-	public override unowned GLib.List<weak GLib.Object> get_objects() {
++	public override unowned GLib.List<GLib.Object> get_objects() {
+ 		return this.objects;
+ 	}
+
+diff --git a/pkcs11/pkcs11-token.vala b/pkcs11/pkcs11-token.vala
+index 927118c2..c4a83c4e 100644
+--- a/pkcs11/pkcs11-token.vala
++++ b/pkcs11/pkcs11-token.vala
+@@ -208,7 +208,7 @@ public class Token : GLib.Object, Gcr.Collection, Place, Lockable {
+ 	public void remove_object(Gck.Object object) {
+ 		GLib.List<Gck.Object> objects = null;
+ 		objects.append(object);
+-		remove_objects(objects);
++		remove_objects(objects.copy());
+ 	}
+
+ 	public bool has_mechanism(ulong mechanism) {
+@@ -387,7 +387,7 @@ public class Token : GLib.Object, Gcr.Collection, Place, Lockable {
+ 		update_visibility(show, true);
+ 	}
+
+-	private void remove_objects(GLib.List<GLib.Object> objects) {
++	private void remove_objects(GLib.List<weak GLib.Object> objects) {
+ 		var depaired = new GLib.List<GLib.Object>();
+ 		var hide = new GLib.List<GLib.Object>();
+
+diff --git a/ssh/deleter.vala b/ssh/deleter.vala
+index 10788c25..e2e1221b 100644
+--- a/ssh/deleter.vala
++++ b/ssh/deleter.vala
+@@ -64,7 +64,7 @@ public class Seahorse.Ssh.Deleter : Seahorse.Deleter {
+         return dialog;
+     }
+
+-    public override unowned List<weak GLib.Object> get_objects () {
++    public override unowned GLib.List<GLib.Object> get_objects () {
+         return this.keys;
+     }
+
+diff --git a/ssh/exporter.vala b/ssh/exporter.vala
+index f2d70b4f..e9499493 100644
+--- a/ssh/exporter.vala
++++ b/ssh/exporter.vala
+@@ -97,7 +97,7 @@ public class Seahorse.Ssh.Exporter : GLib.Object, Seahorse.Exporter {
+         }
+     }
+
+-    public unowned GLib.List<weak GLib.Object> get_objects() {
++    public unowned GLib.List<GLib.Object> get_objects() {
+         return this.objects;
+     }
+
+--
+2.22.0
-- 
2.23.0


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

* [bug#37693] Seahorse build
  2019-10-10  8:56 [bug#37693] Seahorse build Leo Prikler
@ 2019-10-13 12:35 ` Leo Prikler
  2019-10-15 12:09 ` Ludovic Courtès
                   ` (2 subsequent siblings)
  3 siblings, 0 replies; 9+ messages in thread
From: Leo Prikler @ 2019-10-13 12:35 UTC (permalink / raw)
  To: 37693

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

This is a fixup to the previous patch, which adds the patches to
gnu/local.mk.

Unfortunately, the message I've sent along with the previous patch went
missing, so I'm trying to reconstruct it here:
"This patch fixes the currently broken Seahorse build. Seahorse has
switched to the Meson build system upstream, so this change is
reflected in the package and inputs were adjusted. Two upstream patches
are also applied, one of them with adjusted offset."

[-- Attachment #2: 0002-gnu-add-seahorse-patches-to-local.mk.patch --]
[-- Type: text/x-patch, Size: 959 bytes --]

From 9fcae57f90d48249a274482e888982588fda6b9f Mon Sep 17 00:00:00 2001
From: Leo Prikler <leo.prikler@student.tugraz.at>
Date: Sun, 13 Oct 2019 14:29:56 +0200
Subject: [PATCH 2/2] gnu: add seahorse patches to local.mk

* /gnu/local.mk: Add seahorse patches.
---
 gnu/local.mk | 2 ++
 1 file changed, 2 insertions(+)

diff --git a/gnu/local.mk b/gnu/local.mk
index 0a4ee7c986..9e3687b075 100644
--- a/gnu/local.mk
+++ b/gnu/local.mk
@@ -1315,6 +1315,8 @@ dist_patch_DATA =						\
   %D%/packages/patches/scotch-build-parallelism.patch		\
   %D%/packages/patches/scotch-integer-declarations.patch	\
   %D%/packages/patches/sdl-libx11-1.6.patch			\
+  %D%/packages/patches/seahorse-fix-libsecret-breakage.patch	\
+  %D%/packages/patches/seahorse-fix-type-mismatches.patch	\
   %D%/packages/patches/seq24-rename-mutex.patch			\
   %D%/packages/patches/sharutils-CVE-2018-1000097.patch		\
   %D%/packages/patches/shishi-fix-libgcrypt-detection.patch	\
-- 
2.23.0


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

* [bug#37693] Seahorse build
  2019-10-10  8:56 [bug#37693] Seahorse build Leo Prikler
  2019-10-13 12:35 ` Leo Prikler
@ 2019-10-15 12:09 ` Ludovic Courtès
  2019-10-15 12:36   ` Leo Prikler
  2019-10-21 15:25 ` Miguel Arruga Vivas
  2019-10-21 16:59 ` Miguel
  3 siblings, 1 reply; 9+ messages in thread
From: Ludovic Courtès @ 2019-10-15 12:09 UTC (permalink / raw)
  To: Leo Prikler; +Cc: 37693

Hi Leo,

Leo Prikler <leo.prikler@student.tugraz.at> skribis:

> From e2b17f74841ae14ac96782539ec8bfc1c639ed34 Mon Sep 17 00:00:00 2001
> From: Leo Prikler <leo.prikler@student.tugraz.at>
> Date: Thu, 10 Oct 2019 10:16:47 +0200
> Subject: [PATCH] gnu: seahorse: Fix build
>
> * /gnu/packages/gnome.scm: (seahorse): Use meson-build-system.
> Update inputs. Add patches to fix compile errors.
> * /gnu/packages/patches/seahorse-fix-libsecret-breakage.patch: New file.
> * /gnu/packages/patches/seahorse-fix-type-mismatches.patch: New file.

Great that you were able to fix it!

> -    (build-system glib-or-gtk-build-system)
> +         "1sbj1czlx1fakm72dwgbn0bwm12j838yaky4mkf6hf8j8afnxmzp"))
> +       (patches
> +        (search-patches "seahorse-fix-libsecret-breakage.patch"
> +                        "seahorse-fix-type-mismatches.patch"))))
> +    (build-system meson-build-system)

Is changing the build system necessary to fix the problems you
encountered, or is it a side change?  From the current build log at
<https://ci.guix.gnu.org/log/3j08dl90hlyq9haf9lh61k43fs60dcih-seahorse-3.30>,
it does seem that the answer is “yes”.  :-)

>      (inputs
> -     `(("gtk+" ,gtk+)
> +     `(("avahi" ,avahi)
> +       ("gtk+" ,gtk+)
>         ("gcr" ,gcr)
>         ("gnupg" ,gnupg)
>         ("gpgme" ,gpgme)
> @@ -2244,8 +2248,10 @@ engineering.")
>      (native-inputs
>       `(("intltool" ,intltool)
>         ("glib:bin" ,glib "bin")
> +       ("gtk+:bin" ,gtk+ "bin")
>         ("itstool" ,itstool)
>         ("pkg-config" ,pkg-config)
> +       ("vala" ,vala)

Likewise, these changes were necessary, right?

> diff --git a/gnu/packages/patches/seahorse-fix-libsecret-breakage.patch b/gnu/packages/patches/seahorse-fix-libsecret-breakage.patch
> new file mode 100644
> index 0000000000..d6396c227a
> --- /dev/null
> +++ b/gnu/packages/patches/seahorse-fix-libsecret-breakage.patch

[...]

> --- /dev/null
> +++ b/gnu/packages/patches/seahorse-fix-type-mismatches.patch
> @@ -0,0 +1,154 @@
> +From 12178d7cc7fad52f61003b3221bd0a713ef80507 Mon Sep 17 00:00:00 2001
> +From: Rico Tzschichholz <ricotz@ubuntu.com>
> +Date: Mon, 5 Nov 2018 09:59:27 +0100
> +Subject: [PATCH] Fix type-argument mismatches related to GLib.List

One last thing: are those patches upstream, or is there a bug report
where they were discussed?

If so could you share the URLs?  I can add them on your behalf in the
.patch file if you want.  That’ll allow us to better keep track of where
our patches come from and when we can remove them.

Thanks!

Ludo’.

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

* [bug#37693] Seahorse build
  2019-10-15 12:09 ` Ludovic Courtès
@ 2019-10-15 12:36   ` Leo Prikler
  0 siblings, 0 replies; 9+ messages in thread
From: Leo Prikler @ 2019-10-15 12:36 UTC (permalink / raw)
  To: Ludovic Courtès; +Cc: 37693

Am Dienstag, den 15.10.2019, 14:09 +0200 schrieb Ludovic Courtès:
> > -    (build-system glib-or-gtk-build-system)
> > +         "1sbj1czlx1fakm72dwgbn0bwm12j838yaky4mkf6hf8j8afnxmzp"))
> > +       (patches
> > +        (search-patches "seahorse-fix-libsecret-breakage.patch"
> > +                        "seahorse-fix-type-mismatches.patch"))))
> > +    (build-system meson-build-system)
> 
> Is changing the build system necessary to fix the problems you
> encountered, or is it a side change?  From the current build log at
> <
> https://ci.guix.gnu.org/log/3j08dl90hlyq9haf9lh61k43fs60dcih-seahorse-3.30>
> ;,
> it does seem that the answer is “yes”.  :-)
Yes, Seahorse 3.30 no longer builds using Autotools.

> >      (inputs
> > -     `(("gtk+" ,gtk+)
> > +     `(("avahi" ,avahi)
> > +       ("gtk+" ,gtk+)
> >         ("gcr" ,gcr)
> >         ("gnupg" ,gnupg)
> >         ("gpgme" ,gpgme)
> > @@ -2244,8 +2248,10 @@ engineering.")
> >      (native-inputs
> >       `(("intltool" ,intltool)
> >         ("glib:bin" ,glib "bin")
> > +       ("gtk+:bin" ,gtk+ "bin")
> >         ("itstool" ,itstool)
> >         ("pkg-config" ,pkg-config)
> > +       ("vala" ,vala)
> 
> Likewise, these changes were necessary, right?

Indeed. After changing the build system, I've repeatedly run "guix
build seahorse" and added the inputs that the build system complained
about until the build succeeded. ("gtk+:bin" is necessary due to the
icon cache IIRC).

> One last thing: are those patches upstream, or is there a bug report
> where they were discussed?
> 
> If so could you share the URLs?  I can add them on your behalf in the
> .patch file if you want.  That’ll allow us to better keep track of
> where
> our patches come from and when we can remove them.

These patches come from Seahorse upstream. 
seahorse-fix-libsecret-breakage.patch was taken from [1], with its
offset adjusted.
seahorse-fix-type-mismatches.patch was taken from [2] as-is.
The latter appears to be fixed in 3.32 (and also some versions prior),
the former in 3.34.

All the best,
Leo

[1] 
https://gitlab.gnome.org/GNOME/seahorse/commit/d9db29db567012b7c72e85e1be1fbf55fcc9b667
[2] https://gitlab.gnome.org/GNOME/seahorse/merge_requests/31

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

* [bug#37693] Seahorse build
  2019-10-10  8:56 [bug#37693] Seahorse build Leo Prikler
  2019-10-13 12:35 ` Leo Prikler
  2019-10-15 12:09 ` Ludovic Courtès
@ 2019-10-21 15:25 ` Miguel Arruga Vivas
  2019-10-21 16:22   ` Leo Prikler
  2019-10-21 16:59 ` Miguel
  3 siblings, 1 reply; 9+ messages in thread
From: Miguel Arruga Vivas @ 2019-10-21 15:25 UTC (permalink / raw)
  To: 37693; +Cc: Ludovic Courtès, Leo Prikler

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

Hi,

I've hit the same issue, although I tacked it differently because I
updated to 3.30.1.1.  As I didn't knew about this patch, I did it by
myself, but I wouldn't upload it without changing the authorship.

Tue, 15 Oct 2019 14:36:01 +0200 Leo Prikler:
> Am Dienstag, den 15.10.2019, 14:09 +0200 schrieb Ludovic Courtès:
> > > -    (build-system glib-or-gtk-build-system)
> > > +         "1sbj1czlx1fakm72dwgbn0bwm12j838yaky4mkf6hf8j8afnxmzp"))
> > > +       (patches
> > > +        (search-patches "seahorse-fix-libsecret-breakage.patch"
> > > +                        "seahorse-fix-type-mismatches.patch"))))
> > > +    (build-system meson-build-system)
> > 
> > Is changing the build system necessary to fix the problems you
> > encountered, or is it a side change?  From the current build log at
> > <
> > https://ci.guix.gnu.org/log/3j08dl90hlyq9haf9lh61k43fs60dcih-seahorse-3.30>
> > ;,
> > it does seem that the answer is “yes”.  :-)
> Yes, Seahorse 3.30 no longer builds using Autotools.

I agree.

> > >      (inputs
> > > -     `(("gtk+" ,gtk+)
> > > +     `(("avahi" ,avahi)
> > > +       ("gtk+" ,gtk+)
> > >         ("gcr" ,gcr)
> > >         ("gnupg" ,gnupg)
> > >         ("gpgme" ,gpgme)
> > > @@ -2244,8 +2248,10 @@ engineering.")
> > >      (native-inputs
> > >       `(("intltool" ,intltool)
> > >         ("glib:bin" ,glib "bin")
> > > +       ("gtk+:bin" ,gtk+ "bin")
> > >         ("itstool" ,itstool)
> > >         ("pkg-config" ,pkg-config)
> > > +       ("vala" ,vala)
> > 
> > Likewise, these changes were necessary, right?
> 
> Indeed. After changing the build system, I've repeatedly run "guix
> build seahorse" and added the inputs that the build system complained
> about until the build succeeded. ("gtk+:bin" is necessary due to the
> icon cache IIRC).

I've solved this modifying the %standard-phases, as in other packages
from gnome.

> > One last thing: are those patches upstream, or is there a bug report
> > where they were discussed?
> > 
> > If so could you share the URLs?  I can add them on your behalf in
> > > the
> > .patch file if you want.  That’ll allow us to better keep track of
> > where
> > our patches come from and when we can remove them.
> 
> These patches come from Seahorse upstream. 
> seahorse-fix-libsecret-breakage.patch was taken from [1], with its
> offset adjusted.
> seahorse-fix-type-mismatches.patch was taken from [2] as-is.
> The latter appears to be fixed in 3.32 (and also some versions prior),
> the former in 3.34.
> 
> All the best,
> Leo
> 
> [1] 
> https://gitlab.gnome.org/GNOME/seahorse/commit/d9db29db567012b7c72e85e1be1fbf55fcc9b667
> [2] https://gitlab.gnome.org/GNOME/seahorse/merge_requests/31

I don't have the last one because it is fixed with 3.30.1.1.  Also, I
have an extra python that could be a leftover.

Best regards,
Miguel

[-- Attachment #2: 0001-gnu-seahorse-Update-to-3.30.1.1.patch --]
[-- Type: text/x-patch, Size: 4356 bytes --]

From b91e64a997ab283a390f63c46a4665404fa33803 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Miguel=20=C3=81ngel=20Arruga=20Vivas?=
 <rosen644835@gmail.com>
Date: Sat, 19 Oct 2019 12:04:55 +0200
Subject: [PATCH] gnu: seahorse: Update to 3.30.1.1.

* gnu/packages/gnome.scm (seahorse)[version]: Update version number.
[origin]: Update hash.  Add patch needed for compilation with our
version of libsecret (0.19.1).
* gnu/packages/patches/seahorse-gkr-use-0-on-empty-flags.patch: New
file retrieved from the upstream version control system
https://gitlab.gnome.org/GNOME/seahorse/commit/d9db29db567012b7c72e85e1be1fbf55fcc9b667
---
 gnu/packages/gnome.scm                        | 22 +++++++++++---
 .../seahorse-gkr-use-0-on-empty-flags.patch   | 30 +++++++++++++++++++
 2 files changed, 48 insertions(+), 4 deletions(-)
 create mode 100644 gnu/packages/patches/seahorse-gkr-use-0-on-empty-flags.patch

diff --git a/gnu/packages/gnome.scm b/gnu/packages/gnome.scm
index fafb93be08..4a759dbd1b 100644
--- a/gnu/packages/gnome.scm
+++ b/gnu/packages/gnome.scm
@@ -2223,7 +2223,7 @@ engineering.")
 (define-public seahorse
   (package
     (name "seahorse")
-    (version "3.30")
+    (version "3.30.1.1")
     (source
      (origin
        (method url-fetch)
@@ -2232,8 +2232,19 @@ engineering.")
                            version ".tar.xz"))
        (sha256
         (base32
-         "1sbj1czlx1fakm72dwgbn0bwm12j838yaky4mkf6hf8j8afnxmzp"))))
-    (build-system glib-or-gtk-build-system)
+         "12x7xmwh62yl0ax90v8nkx3jqzviaz9hz2g56yml78wzww20gawy"))
+       (patches (search-patches
+                 "seahorse-gkr-use-0-on-empty-flags.patch"))))
+    (build-system meson-build-system)
+    (arguments
+     '(#:phases
+       (modify-phases %standard-phases
+         (add-after 'unpack 'skip-gtk-update-icon-cache
+           ;; Don't create 'icon-theme.cache'.
+           (lambda _
+             (substitute* "build-aux/meson_post_install.py"
+               (("gtk-update-icon-cache") "true"))
+             #t)))))
     (inputs
      `(("gtk+" ,gtk+)
        ("gcr" ,gcr)
@@ -2241,13 +2252,16 @@ engineering.")
        ("gpgme" ,gpgme)
        ("openldap" ,openldap)
        ("openssh" ,openssh)
+       ("avahi" ,avahi)
        ("libsecret" ,libsecret)
        ("libsoup" ,libsoup)))
     (native-inputs
-     `(("intltool" ,intltool)
+     `(("gettext" ,gettext-minimal)
        ("glib:bin" ,glib "bin")
        ("itstool" ,itstool)
        ("pkg-config" ,pkg-config)
+       ("python" ,python)
+       ("vala" ,vala)
        ("xmllint" ,libxml2)))
     (home-page "https://launchpad.net/gnome-themes-standard")
     (synopsis "Manage encryption keys and passwords in the GNOME keyring")
diff --git a/gnu/packages/patches/seahorse-gkr-use-0-on-empty-flags.patch b/gnu/packages/patches/seahorse-gkr-use-0-on-empty-flags.patch
new file mode 100644
index 0000000000..e236f6a2e1
--- /dev/null
+++ b/gnu/packages/patches/seahorse-gkr-use-0-on-empty-flags.patch
@@ -0,0 +1,30 @@
+From d9db29db567012b7c72e85e1be1fbf55fcc9b667 Mon Sep 17 00:00:00 2001
+From: Niels De Graef <nielsdegraef@gmail.com>
+Date: Sat, 11 May 2019 09:02:34 +0200
+Subject: [PATCH] gkr: Use 0 on empty flags
+
+A Flags-type variable without any flag set can be replaced with 0, so
+this is a safe thing to do. It also prevents us from having to deal with
+the accidental API break in libsecret (see
+https://gitlab.gnome.org/GNOME/libsecret/merge_requests/19)
+---
+ gkr/gkr-keyring-add.vala | 3 +--
+ 1 file changed, 1 insertion(+), 2 deletions(-)
+
+diff --git a/gkr/gkr-keyring-add.vala b/gkr/gkr-keyring-add.vala
+index 4e92a520..f60c9a22 100644
+--- a/gkr/gkr-keyring-add.vala
++++ b/gkr/gkr-keyring-add.vala
+@@ -41,8 +41,7 @@ public class Seahorse.Gkr.KeyringAdd : Gtk.Dialog {
+ 
+         var cancellable = Dialog.begin_request(this);
+         var service = Backend.instance().service;
+-        Secret.Collection.create.begin(service, this.name_entry.text, null,
+-                                       Secret.CollectionCreateFlags.COLLECTION_CREATE_NONE,
++        Secret.Collection.create.begin(service, this.name_entry.text, null, 0,
+                                        cancellable, (obj, res) => {
+             /* Clear the operation without cancelling it since it is complete */
+             Dialog.complete_request(this, false);
+-- 
+2.23.0
+
-- 
2.23.0


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

* [bug#37693] Seahorse build
  2019-10-21 15:25 ` Miguel Arruga Vivas
@ 2019-10-21 16:22   ` Leo Prikler
  0 siblings, 0 replies; 9+ messages in thread
From: Leo Prikler @ 2019-10-21 16:22 UTC (permalink / raw)
  To: Miguel Arruga Vivas, 37693; +Cc: Ludovic Courtès

Am Montag, den 21.10.2019, 17:25 +0200 schrieb Miguel Arruga Vivas:
> > > >      (inputs
> > > > -     `(("gtk+" ,gtk+)
> > > > +     `(("avahi" ,avahi)
> > > > +       ("gtk+" ,gtk+)
> > > >         ("gcr" ,gcr)
> > > >         ("gnupg" ,gnupg)
> > > >         ("gpgme" ,gpgme)
> > > > @@ -2244,8 +2248,10 @@ engineering.")
> > > >      (native-inputs
> > > >       `(("intltool" ,intltool)
> > > >         ("glib:bin" ,glib "bin")
> > > > +       ("gtk+:bin" ,gtk+ "bin")
> > > >         ("itstool" ,itstool)
> > > >         ("pkg-config" ,pkg-config)
> > > > +       ("vala" ,vala)
> > > 
> > > Likewise, these changes were necessary, right?
> > 
> > Indeed. After changing the build system, I've repeatedly run "guix
> > build seahorse" and added the inputs that the build system
> > complained
> > about until the build succeeded. ("gtk+:bin" is necessary due to
> > the
> > icon cache IIRC).
> 
> I've solved this modifying the %standard-phases, as in other packages
> from gnome.
I'm not sure, whether that is the right approach. Seahorse used glib-
or-gtk-build-system before, so I would assume the generation of those
caches is expected.

> I don't have the last one because it is fixed with 3.30.1.1.  Also, I
> have an extra python that could be a leftover.
Would you mind checking, whether your patch works with gtk+:bin added
and without python? If so, I'd find it preferable to mine.

Regards,

Leo

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

* [bug#37693] Seahorse build
  2019-10-10  8:56 [bug#37693] Seahorse build Leo Prikler
                   ` (2 preceding siblings ...)
  2019-10-21 15:25 ` Miguel Arruga Vivas
@ 2019-10-21 16:59 ` Miguel
  2019-10-21 17:20   ` Miguel Arruga Vivas
  3 siblings, 1 reply; 9+ messages in thread
From: Miguel @ 2019-10-21 16:59 UTC (permalink / raw)
  To: Leo Prikler; +Cc: Ludovic Courtès, 37693

Hi,

El 21 oct. 2019 18:22, Leo Prikler <leo.prikler@student.tugraz.at> escribió:
>
> Am Montag, den 21.10.2019, 17:25 +0200 schrieb Miguel Arruga Vivas: 
>>> > > > > @@ -2244,8 +2248,10 @@ engineering.") 
> > > > >      (native-inputs 
> > > > >       `(("intltool" ,intltool) 
> > > > >         ("glib:bin" ,glib "bin") 
> > > > > +       ("gtk+:bin" ,gtk+ "bin") 
> > > > >         ("itstool" ,itstool) 
> > > > >         ("pkg-config" ,pkg-config) 
> > > > > +       ("vala" ,vala) 
> > > > 
> > > > Likewise, these changes were necessary, right? 
> > > 
> > > Indeed. After changing the build system, I've repeatedly run "guix 
> > > build seahorse" and added the inputs that the build system 
> > > complained 
> > > about until the build succeeded. ("gtk+:bin" is necessary due to 
> > > the 
> > > icon cache IIRC). 
> > 
> > I've solved this modifying the %standard-phases, as in other packages 
> > from gnome. 
> I'm not sure, whether that is the right approach. Seahorse used glib- 
> or-gtk-build-system before, so I would assume the generation of those 
> caches is expected. 

The icon.cache is generated later by a profile hook, as it must include all icons from the user-installed packages and not only seahorse ones. That's the reason behind its removal during installation phase.

> > I don't have the last one because it is fixed with 3.30.1.1.  Also, I 
> > have an extra python that could be a leftover. 
> Would you mind checking, whether your patch works with gtk+:bin added 
> and without python? If so, I'd find it preferable to mine. 

I cannot test the python removal right now, but I'll answer as soon I can build it.

Best regards,
Miguel

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

* [bug#37693] Seahorse build
  2019-10-21 16:59 ` Miguel
@ 2019-10-21 17:20   ` Miguel Arruga Vivas
  2019-10-22 16:27     ` bug#37693: " Ludovic Courtès
  0 siblings, 1 reply; 9+ messages in thread
From: Miguel Arruga Vivas @ 2019-10-21 17:20 UTC (permalink / raw)
  To: Leo Prikler; +Cc: Ludovic Courtès, 37693

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

El Mon, 21 Oct 2019 18:59:39 +0200
Miguel <rosen644835@gmail.com> escribió:
> I cannot test the python removal right now, but I'll answer as soon I
> can build it.

I confirm it works with the attached patch (without python as an
explicit dependency).

Best regards,
Miguel

[-- Attachment #2: 0001-gnu-seahorse-Update-to-3.30.1.1.patch --]
[-- Type: text/x-patch, Size: 4466 bytes --]

From c2cbf056d277842b79a5e4588ab38980ef9d8db9 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Miguel=20=C3=81ngel=20Arruga=20Vivas?=
 <rosen644835@gmail.com>
Date: Mon, 21 Oct 2019 19:16:24 +0200
Subject: [PATCH] gnu: seahorse: Update to 3.30.1.1.

* gnu/packages/gnome.scm (seahorse)[version]: Update version number.
[origin]: Update hash.  Add patch needed for compilation with our
version of libsecret (0.19.1).
[inputs]: Add avahi dependency.
[native-inputs]: Add vala dependency.  Use gettext-minimal instead of
intltool, as only xgettext is used.
* gnu/packages/patches/seahorse-gkr-use-0-on-empty-flags.patch: New
file retrieved from the upstream version control system
https://gitlab.gnome.org/GNOME/seahorse/commit/d9db29db567012b7c72e85e1be1fbf55fcc9b667
---
 gnu/packages/gnome.scm                        | 21 ++++++++++---
 .../seahorse-gkr-use-0-on-empty-flags.patch   | 30 +++++++++++++++++++
 2 files changed, 47 insertions(+), 4 deletions(-)
 create mode 100644 gnu/packages/patches/seahorse-gkr-use-0-on-empty-flags.patch

diff --git a/gnu/packages/gnome.scm b/gnu/packages/gnome.scm
index fafb93be08..5cc915b4a9 100644
--- a/gnu/packages/gnome.scm
+++ b/gnu/packages/gnome.scm
@@ -2223,7 +2223,7 @@ engineering.")
 (define-public seahorse
   (package
     (name "seahorse")
-    (version "3.30")
+    (version "3.30.1.1")
     (source
      (origin
        (method url-fetch)
@@ -2232,8 +2232,19 @@ engineering.")
                            version ".tar.xz"))
        (sha256
         (base32
-         "1sbj1czlx1fakm72dwgbn0bwm12j838yaky4mkf6hf8j8afnxmzp"))))
-    (build-system glib-or-gtk-build-system)
+         "12x7xmwh62yl0ax90v8nkx3jqzviaz9hz2g56yml78wzww20gawy"))
+       (patches (search-patches
+                 "seahorse-gkr-use-0-on-empty-flags.patch"))))
+    (build-system meson-build-system)
+    (arguments
+     '(#:phases
+       (modify-phases %standard-phases
+         (add-after 'unpack 'skip-gtk-update-icon-cache
+           ;; Don't create 'icon-theme.cache'.
+           (lambda _
+             (substitute* "build-aux/meson_post_install.py"
+               (("gtk-update-icon-cache") "true"))
+             #t)))))
     (inputs
      `(("gtk+" ,gtk+)
        ("gcr" ,gcr)
@@ -2241,13 +2252,15 @@ engineering.")
        ("gpgme" ,gpgme)
        ("openldap" ,openldap)
        ("openssh" ,openssh)
+       ("avahi" ,avahi)
        ("libsecret" ,libsecret)
        ("libsoup" ,libsoup)))
     (native-inputs
-     `(("intltool" ,intltool)
+     `(("gettext" ,gettext-minimal)
        ("glib:bin" ,glib "bin")
        ("itstool" ,itstool)
        ("pkg-config" ,pkg-config)
+       ("vala" ,vala)
        ("xmllint" ,libxml2)))
     (home-page "https://launchpad.net/gnome-themes-standard")
     (synopsis "Manage encryption keys and passwords in the GNOME keyring")
diff --git a/gnu/packages/patches/seahorse-gkr-use-0-on-empty-flags.patch b/gnu/packages/patches/seahorse-gkr-use-0-on-empty-flags.patch
new file mode 100644
index 0000000000..e236f6a2e1
--- /dev/null
+++ b/gnu/packages/patches/seahorse-gkr-use-0-on-empty-flags.patch
@@ -0,0 +1,30 @@
+From d9db29db567012b7c72e85e1be1fbf55fcc9b667 Mon Sep 17 00:00:00 2001
+From: Niels De Graef <nielsdegraef@gmail.com>
+Date: Sat, 11 May 2019 09:02:34 +0200
+Subject: [PATCH] gkr: Use 0 on empty flags
+
+A Flags-type variable without any flag set can be replaced with 0, so
+this is a safe thing to do. It also prevents us from having to deal with
+the accidental API break in libsecret (see
+https://gitlab.gnome.org/GNOME/libsecret/merge_requests/19)
+---
+ gkr/gkr-keyring-add.vala | 3 +--
+ 1 file changed, 1 insertion(+), 2 deletions(-)
+
+diff --git a/gkr/gkr-keyring-add.vala b/gkr/gkr-keyring-add.vala
+index 4e92a520..f60c9a22 100644
+--- a/gkr/gkr-keyring-add.vala
++++ b/gkr/gkr-keyring-add.vala
+@@ -41,8 +41,7 @@ public class Seahorse.Gkr.KeyringAdd : Gtk.Dialog {
+ 
+         var cancellable = Dialog.begin_request(this);
+         var service = Backend.instance().service;
+-        Secret.Collection.create.begin(service, this.name_entry.text, null,
+-                                       Secret.CollectionCreateFlags.COLLECTION_CREATE_NONE,
++        Secret.Collection.create.begin(service, this.name_entry.text, null, 0,
+                                        cancellable, (obj, res) => {
+             /* Clear the operation without cancelling it since it is complete */
+             Dialog.complete_request(this, false);
+-- 
+2.23.0
+
-- 
2.23.0


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

* bug#37693: Seahorse build
  2019-10-21 17:20   ` Miguel Arruga Vivas
@ 2019-10-22 16:27     ` Ludovic Courtès
  0 siblings, 0 replies; 9+ messages in thread
From: Ludovic Courtès @ 2019-10-22 16:27 UTC (permalink / raw)
  To: Miguel Arruga Vivas; +Cc: Leo Prikler, 37693-done

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

Hello,

Miguel Arruga Vivas <rosen644835@gmail.com> skribis:

>>From c2cbf056d277842b79a5e4588ab38980ef9d8db9 Mon Sep 17 00:00:00 2001
> From: =?UTF-8?q?Miguel=20=C3=81ngel=20Arruga=20Vivas?=
>  <rosen644835@gmail.com>
> Date: Mon, 21 Oct 2019 19:16:24 +0200
> Subject: [PATCH] gnu: seahorse: Update to 3.30.1.1.
>
> * gnu/packages/gnome.scm (seahorse)[version]: Update version number.
> [origin]: Update hash.  Add patch needed for compilation with our
> version of libsecret (0.19.1).
> [inputs]: Add avahi dependency.
> [native-inputs]: Add vala dependency.  Use gettext-minimal instead of
> intltool, as only xgettext is used.
> * gnu/packages/patches/seahorse-gkr-use-0-on-empty-flags.patch: New
> file retrieved from the upstream version control system
> https://gitlab.gnome.org/GNOME/seahorse/commit/d9db29db567012b7c72e85e1be1fbf55fcc9b667

Wonderful.  Applied with the changes below.

Thanks to the two of you!  :-)

Ludo’.


[-- Warning: decoded text below may be mangled, UTF-8 assumed --]
[-- Attachment #2: Type: text/x-patch, Size: 1157 bytes --]

diff --git a/gnu/local.mk b/gnu/local.mk
index 4292e6f415..611761c2c9 100644
--- a/gnu/local.mk
+++ b/gnu/local.mk
@@ -1324,6 +1324,7 @@ dist_patch_DATA =						\
   %D%/packages/patches/scotch-build-parallelism.patch		\
   %D%/packages/patches/scotch-integer-declarations.patch	\
   %D%/packages/patches/sdl-libx11-1.6.patch			\
+  %D%/packages/patches/seahorse-gkr-use-0-on-empty-flags.patch	\
   %D%/packages/patches/seq24-rename-mutex.patch			\
   %D%/packages/patches/sharutils-CVE-2018-1000097.patch		\
   %D%/packages/patches/shishi-fix-libgcrypt-detection.patch	\
diff --git a/gnu/packages/patches/seahorse-gkr-use-0-on-empty-flags.patch b/gnu/packages/patches/seahorse-gkr-use-0-on-empty-flags.patch
index e236f6a2e1..55866d2973 100644
--- a/gnu/packages/patches/seahorse-gkr-use-0-on-empty-flags.patch
+++ b/gnu/packages/patches/seahorse-gkr-use-0-on-empty-flags.patch
@@ -1,3 +1,5 @@
+Patch from <https://gitlab.gnome.org/GNOME/seahorse/commit/d9db29db567012b7c72e85e1be1fbf55fcc9b667>.
+
 From d9db29db567012b7c72e85e1be1fbf55fcc9b667 Mon Sep 17 00:00:00 2001
 From: Niels De Graef <nielsdegraef@gmail.com>
 Date: Sat, 11 May 2019 09:02:34 +0200

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

end of thread, other threads:[~2019-10-22 16:29 UTC | newest]

Thread overview: 9+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2019-10-10  8:56 [bug#37693] Seahorse build Leo Prikler
2019-10-13 12:35 ` Leo Prikler
2019-10-15 12:09 ` Ludovic Courtès
2019-10-15 12:36   ` Leo Prikler
2019-10-21 15:25 ` Miguel Arruga Vivas
2019-10-21 16:22   ` Leo Prikler
2019-10-21 16:59 ` Miguel
2019-10-21 17:20   ` Miguel Arruga Vivas
2019-10-22 16:27     ` bug#37693: " Ludovic Courtès

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