unofficial mirror of guix-patches@gnu.org 
 help / color / mirror / code / Atom feed
* [bug#53161] [PATCH 0/2] gnu: eog: Fix failing build due to libportal upgrade
@ 2022-01-10 15:32 Daniel Meißner via Guix-patches via
  2022-01-10 15:37 ` [bug#53161] [PATCH 1/2] gnu: libportal: Build backend for GTK+3 Daniel Meißner via Guix-patches via
  2022-01-11 15:19 ` bug#53161: [PATCH 0/2] gnu: eog: Fix failing build due to libportal upgrade Guillaume Le Vaillant
  0 siblings, 2 replies; 4+ messages in thread
From: Daniel Meißner via Guix-patches via @ 2022-01-10 15:32 UTC (permalink / raw)
  To: 53161; +Cc: Daniel Meißner

Hi Guix,

after upgrading libportal to 0.5 eog fails to build.  This little patch series
fixes this.

Thanks,
Daniel

Daniel Meißner (2):
  gnu: libportal: Build backend for GTK+3
  gnu: eog: Fix build with libportal-0.5

 gnu/local.mk                                  |  2 +
 gnu/packages/freedesktop.scm                  |  3 +-
 gnu/packages/gnome.scm                        |  5 ++-
 .../patches/eog-update-libportal-usage.patch  | 40 +++++++++++++++++++
 4 files changed, 48 insertions(+), 2 deletions(-)
 create mode 100644 gnu/packages/patches/eog-update-libportal-usage.patch

-- 
2.34.0





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

* [bug#53161] [PATCH 1/2] gnu: libportal: Build backend for GTK+3
  2022-01-10 15:32 [bug#53161] [PATCH 0/2] gnu: eog: Fix failing build due to libportal upgrade Daniel Meißner via Guix-patches via
@ 2022-01-10 15:37 ` Daniel Meißner via Guix-patches via
  2022-01-10 15:37   ` [bug#53161] [PATCH 2/2] gnu: eog: Fix build with libportal-0.5 Daniel Meißner via Guix-patches via
  2022-01-11 15:19 ` bug#53161: [PATCH 0/2] gnu: eog: Fix failing build due to libportal upgrade Guillaume Le Vaillant
  1 sibling, 1 reply; 4+ messages in thread
From: Daniel Meißner via Guix-patches via @ 2022-01-10 15:37 UTC (permalink / raw)
  To: 53161; +Cc: Daniel Meißner

* gnu/packages/freedesktop.scm (libportal)[arguments]: Change configure-flags
to build backend for GTK+3.
---
 gnu/packages/freedesktop.scm | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

diff --git a/gnu/packages/freedesktop.scm b/gnu/packages/freedesktop.scm
index a7d400d3bf..2b0739020d 100644
--- a/gnu/packages/freedesktop.scm
+++ b/gnu/packages/freedesktop.scm
@@ -28,6 +28,7 @@
 ;;; Copyright © 2021 Maxime Devos <maximedevos@telenet.be>
 ;;; Copyright © 2021 John Kehayias <john.kehayias@protonmail.com>
 ;;; Copyright © 2021 Maxim Cournoyer <maxim.cournoyer@gmail.com>
+;;; Copyright © 2022 Daniel Meißner <daniel.meissner-i4k@ruhr-uni-bochum.de>
 ;;;
 ;;; This file is part of GNU Guix.
 ;;;
@@ -2233,7 +2234,7 @@ (define-public libportal
     (build-system meson-build-system)
     (arguments
      `(#:configure-flags
-       (list "-Dbackends=gtk4,qt5"
+       (list "-Dbackends=gtk4,gtk3,qt5"
              "-Ddocs=false")))          ; requires unpackaged gi-docgen
     (native-inputs
      (list pkg-config
-- 
2.34.0





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

* [bug#53161] [PATCH 2/2] gnu: eog: Fix build with libportal-0.5
  2022-01-10 15:37 ` [bug#53161] [PATCH 1/2] gnu: libportal: Build backend for GTK+3 Daniel Meißner via Guix-patches via
@ 2022-01-10 15:37   ` Daniel Meißner via Guix-patches via
  0 siblings, 0 replies; 4+ messages in thread
From: Daniel Meißner via Guix-patches via @ 2022-01-10 15:37 UTC (permalink / raw)
  To: 53161; +Cc: Daniel Meißner

After the update to libportal-0.5 in 294476022f19139e290acb448d4575de0f851673
eog fails with:

../eog-40.3/src/eog-util.c:48:10: fatal error: libportal/portal-gtk3.h: No such file or directory

This was already fixed upstream and this commits extracts the patch.

* gnu/packages/patches/eog-update-libportal-usage.patch: New file.
* gnu/local.mk (dist_patch_DATA): Register it.
* gnu/packages/gnome.scm(eog)[origin]: Add new patch.
---
 gnu/local.mk                                  |  2 +
 gnu/packages/gnome.scm                        |  5 ++-
 .../patches/eog-update-libportal-usage.patch  | 40 +++++++++++++++++++
 3 files changed, 46 insertions(+), 1 deletion(-)
 create mode 100644 gnu/packages/patches/eog-update-libportal-usage.patch

diff --git a/gnu/local.mk b/gnu/local.mk
index fe21cf8062..cc83c87282 100644
--- a/gnu/local.mk
+++ b/gnu/local.mk
@@ -47,6 +47,7 @@
 # Copyright © 2021 Dmitry Polyakov <polyakov@liltechdude.xyz>
 # Copyright © 2021 Andrew Tropin <andrew@trop.in>
 # Copyright © 2021 Simon Tournier <zimon.toutoune@gmail.com>
+# Copyright © 2022 Daniel Meißner <daniel.meissner-i4k@ruhr-uni-bochum.de>
 #
 # This file is part of GNU Guix.
 #
@@ -1027,6 +1028,7 @@ dist_patch_DATA =						\
   %D%/packages/patches/emacs-wordnut-require-adaptive-wrap.patch	\
   %D%/packages/patches/enjarify-setup-py.patch			\
   %D%/packages/patches/enlightenment-fix-setuid-path.patch	\
+  %D%/packages/patches/eog-update-libportal-usage.patch		\
   %D%/packages/patches/erlang-man-path.patch			\
   %D%/packages/patches/esmtp-add-lesmtp.patch		\
   %D%/packages/patches/eudev-rules-directory.patch		\
diff --git a/gnu/packages/gnome.scm b/gnu/packages/gnome.scm
index 429b17c00e..c8e82f811b 100644
--- a/gnu/packages/gnome.scm
+++ b/gnu/packages/gnome.scm
@@ -65,6 +65,7 @@
 ;;; Copyright © 2021 Maxime Devos <maximedevos@telenet.be>
 ;;; Copyright © 2021 Josselin Poiret <josselin.poiret@protonmail.ch>
 ;;; Copyright © 2021 Mathieu Othacehe <othacehe@gnu.org>
+;;; Copyright © 2022 Daniel Meißner <daniel.meissner-i4k@ruhr-uni-bochum.de>
 ;;;
 ;;; This file is part of GNU Guix.
 ;;;
@@ -6427,7 +6428,9 @@ (define-public eog
                                 name "-" version ".tar.xz"))
             (sha256
              (base32
-              "0ddjwcd77nw0rxb5x5bz5hd671m8gya9827p8rsnb58x103kpai8"))))
+              "0ddjwcd77nw0rxb5x5bz5hd671m8gya9827p8rsnb58x103kpai8"))
+            ;; XXX: Remove when upgrading to 42.0
+            (patches (search-patches "eog-update-libportal-usage.patch"))))
    (build-system meson-build-system)
    (arguments
     `(#:meson ,meson-0.59         ;positional arguments error with meson 0.60
diff --git a/gnu/packages/patches/eog-update-libportal-usage.patch b/gnu/packages/patches/eog-update-libportal-usage.patch
new file mode 100644
index 0000000000..220a16ddc4
--- /dev/null
+++ b/gnu/packages/patches/eog-update-libportal-usage.patch
@@ -0,0 +1,40 @@
+Fix eog to work with libportal-0.5
+
+This patch is extracted from upstream, see here
+https://gitlab.gnome.org/GNOME/eog/-/commit/a06e6325907e136678b0bbe7058c25d688034afd
+
+diff --git a/meson.build b/meson.build
+index 9a32e4bb..9d49aa45 100644
+--- a/meson.build
++++ b/meson.build
+@@ -165,10 +165,14 @@ config_h.set('HAVE_EXEMPI', enable_xmp)
+ # xdg-desktop-portal support with libportal (optional)
+ enable_libportal = get_option('libportal')
+ if enable_libportal
+-  libportal_dep = dependency('libportal', version: '>= 0.3', required: false)
++  libportal_dep = dependency('libportal', version: '>= 0.5', required: false)
+   assert(libportal_dep.found() and cc.has_header('libportal/portal.h', dependencies: libportal_dep),
+          'xdg-desktop-portal support requested but library not found. Please use -Dlibportal=false')
+ 
+-  common_deps += libportal_dep
++  libportal_gtk3_dep = dependency('libportal-gtk3', version: '>= 0.5', required: false)
++  assert(libportal_gtk3_dep.found() and cc.has_header('libportal-gtk3/portal-gtk3.h', dependencies: libportal_gtk3_dep),
++         'xdg-desktop-portal support requested but library not found. Please use -Dlibportal=false')
++
++  common_deps += [libportal_dep, libportal_gtk3_dep]
+ endif
+ config_h.set('HAVE_LIBPORTAL', enable_libportal)
+
+diff --git a/src/eog-util.c b/src/eog-util.c
+index 90b9768e..56d23472 100644
+--- a/src/eog-util.c
++++ b/src/eog-util.c
+@@ -45,7 +45,7 @@
+ #include <glib/gi18n.h>
+ #ifdef HAVE_LIBPORTAL
+ #include <libportal/portal.h>
+-#include <libportal/portal-gtk3.h>
++#include <libportal-gtk3/portal-gtk3.h>
+ #endif
+ 
+ void
-- 
2.34.0





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

* bug#53161: [PATCH 0/2] gnu: eog: Fix failing build due to libportal upgrade
  2022-01-10 15:32 [bug#53161] [PATCH 0/2] gnu: eog: Fix failing build due to libportal upgrade Daniel Meißner via Guix-patches via
  2022-01-10 15:37 ` [bug#53161] [PATCH 1/2] gnu: libportal: Build backend for GTK+3 Daniel Meißner via Guix-patches via
@ 2022-01-11 15:19 ` Guillaume Le Vaillant
  1 sibling, 0 replies; 4+ messages in thread
From: Guillaume Le Vaillant @ 2022-01-11 15:19 UTC (permalink / raw)
  To: Daniel Meißner; +Cc: 53161-done

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

Patches pushed as e92afc59048502a4a23ea5866522ba5f3ccc8aba and
following.
Thanks.

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

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

end of thread, other threads:[~2022-01-11 15:20 UTC | newest]

Thread overview: 4+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2022-01-10 15:32 [bug#53161] [PATCH 0/2] gnu: eog: Fix failing build due to libportal upgrade Daniel Meißner via Guix-patches via
2022-01-10 15:37 ` [bug#53161] [PATCH 1/2] gnu: libportal: Build backend for GTK+3 Daniel Meißner via Guix-patches via
2022-01-10 15:37   ` [bug#53161] [PATCH 2/2] gnu: eog: Fix build with libportal-0.5 Daniel Meißner via Guix-patches via
2022-01-11 15:19 ` bug#53161: [PATCH 0/2] gnu: eog: Fix failing build due to libportal upgrade Guillaume Le Vaillant

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