Github messages for voidlinux
 help / color / mirror / Atom feed
From: chrysos349 <chrysos349@users.noreply.github.com>
To: ml@inbox.vuxu.org
Subject: Re: [PR PATCH] [Updated] grilo{,-plugins}: update to 0.3.16
Date: Thu, 29 Feb 2024 06:14:25 +0100	[thread overview]
Message-ID: <20240229051425.AD52F24FC8@inbox.vuxu.org> (raw)
In-Reply-To: <gh-mailinglist-notifications-41a7ca26-5023-4802-975b-f1789d68868e-void-packages-48982@inbox.vuxu.org>

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

There is an updated pull request by chrysos349 against master on the void-packages repository

https://github.com/chrysos349/void-packages grilo
https://github.com/void-linux/void-packages/pull/48982

grilo{,-plugins}: update to 0.3.16
split into a separate pr, according to @oreo639's recommendation (https://github.com/void-linux/void-packages/pull/48752#issuecomment-1968076680).

#### Testing the changes
- I tested the changes in this PR: **YES**

#### Local build testing
- I built this PR locally for my native architecture, (x86_64)
- I built this PR locally for these architectures (if supported. mark crossbuilds):
  - aarch64-musl x
  - armv7l x
  - armv6l-musl x

A patch file from https://github.com/void-linux/void-packages/pull/48982.patch is attached

[-- Warning: decoded text below may be mangled, UTF-8 assumed --]
[-- Attachment #2: github-pr-grilo-48982.patch --]
[-- Type: text/x-diff, Size: 5010 bytes --]

From 87fca5dc86388673eb9df100c06ff80930e9ae84 Mon Sep 17 00:00:00 2001
From: chrysos349 <chrysostom349@gmail.com>
Date: Wed, 28 Feb 2024 08:33:51 +0300
Subject: [PATCH 1/2] grilo: update to 0.3.16

---
 srcpkgs/grilo/patches/fix-pc-file.patch | 19 +++++++++++++++++++
 srcpkgs/grilo/template                  | 22 +++++++++++++++-------
 2 files changed, 34 insertions(+), 7 deletions(-)
 create mode 100644 srcpkgs/grilo/patches/fix-pc-file.patch

diff --git a/srcpkgs/grilo/patches/fix-pc-file.patch b/srcpkgs/grilo/patches/fix-pc-file.patch
new file mode 100644
index 00000000000000..ccfeb92624d3a6
--- /dev/null
+++ b/srcpkgs/grilo/patches/fix-pc-file.patch
@@ -0,0 +1,19 @@
+We patch girdir and typelibdir in gobject-introspection to include pc_sysrootdir
+for cross compiling.
+Avoid using pkg-config variables for girdir_for_pc_file and typelibdir_for_pc_file
+since they include pc_sysrootdir, which cannot be overridden using pkgconfig_define.
+meson needs non-absolute paths in order to correctly generate datadir.
+--- a/meson.build
++++ b/meson.build
+@@ -96,9 +96,9 @@ typelibdir_for_pc_file = join_paths('${libdir}', 'girepository-1.0')
+ gobject_introspection = dependency('gobject-introspection-1.0', required: false)
+ if gobject_introspection.found()
+     girdir = gobject_introspection.get_variable(pkgconfig: 'girdir', pkgconfig_define: ['datadir', datadir])
+-    girdir_for_pc_file = gobject_introspection.get_variable(pkgconfig: 'girdir', pkgconfig_define: ['datadir', '${datadir}'])
++    girdir_for_pc_file = join_paths('${datadir}', 'gir-1.0') 
+     typelibdir = gobject_introspection.get_variable(pkgconfig: 'typelibdir', pkgconfig_define: ['libdir', libdir])
+-    typelibdir_for_pc_file = gobject_introspection.get_variable(pkgconfig: 'typelibdir', pkgconfig_define: ['libdir', '${libdir}'])
++    typelibdir_for_pc_file = join_paths('${libdir}', 'girepository-1.0')
+ endif
+ 
+ vapidir = join_paths(datadir, 'vala', 'vapi')
diff --git a/srcpkgs/grilo/template b/srcpkgs/grilo/template
index e47ec38542a50c..7bcdfe595cf02b 100644
--- a/srcpkgs/grilo/template
+++ b/srcpkgs/grilo/template
@@ -1,25 +1,30 @@
 # Template file for 'grilo'
 pkgname=grilo
-version=0.3.13
+version=0.3.16
 revision=1
 build_style=meson
 build_helper="gir"
 configure_args="$(vopt_bool gir enable-introspection) $(vopt_bool gir enable-vala)
- -Denable-gtk-doc=false"
-hostmakedepends="gettext pkg-config glib-devel $(vopt_if gir vala)"
-makedepends="gtk+3-devel libxml2-devel libsoup-devel liboauth-devel
+ $(vopt_bool gtk_doc enable-gtk-doc)"
+hostmakedepends="gettext pkg-config glib-devel $(vopt_if gir vala)
+ $(vopt_if gtk_doc gtk-doc)"
+makedepends="gtk+3-devel libxml2-devel libsoup3-devel liboauth-devel
  totem-pl-parser-devel"
 short_desc="Framework focused on making media discovery and browsing easy"
 maintainer="Enno Boland <gottox@voidlinux.org>"
 license="LGPL-2.1-or-later"
-homepage="http://live.gnome.org/Grilo"
+homepage="https://wiki.gnome.org/Projects/Grilo"
 distfiles="${GNOME_SITE}/${pkgname}/${version%.*}/${pkgname}-${version}.tar.xz"
-checksum=d14837f22341943ed8a189d9f0827a17016b802d18d0ed080e1413de0fdc927b
+checksum=884580e8c5ece280df23aa63ff5234b7d48988a404df7d6bfccd1e77b473bd96
 
 # Package build options
-build_options="gir"
+build_options="gir gtk_doc"
 build_options_default="gir"
 
+if [ -z "$CROSS_BUILD" ]; then
+	build_options_default+=" gtk_doc"
+fi
+
 grilo-devel_package() {
 	depends="${makedepends} grilo>=${version}_${revision}"
 	short_desc+=" - development files"
@@ -31,5 +36,8 @@ grilo-devel_package() {
 		fi
 		vmove usr/lib/pkgconfig
 		vmove "usr/lib/*.so"
+		if [ "$build_option_gtk_doc" ]; then
+			vmove usr/share/gtk-doc
+		fi
 	}
 }

From 0d24a89858d87a8aec011485d78b7ccf7aa76233 Mon Sep 17 00:00:00 2001
From: chrysos349 <chrysostom349@gmail.com>
Date: Wed, 28 Feb 2024 09:00:33 +0300
Subject: [PATCH 2/2] grilo-plugins: update to 0.3.16

---
 srcpkgs/grilo-plugins/template | 8 ++++----
 1 file changed, 4 insertions(+), 4 deletions(-)

diff --git a/srcpkgs/grilo-plugins/template b/srcpkgs/grilo-plugins/template
index 2d306f8de7e48f..568af47a957b8a 100644
--- a/srcpkgs/grilo-plugins/template
+++ b/srcpkgs/grilo-plugins/template
@@ -1,9 +1,9 @@
 # Template file for 'grilo-plugins'
 pkgname=grilo-plugins
-version=0.3.14
-revision=3
+version=0.3.16
+revision=1
 build_style=meson
-configure_args="-Dgoa=disabled"
+configure_args="-Dgoa=enabled"
 hostmakedepends="pkg-config gettext itstool glib-devel gperf tracker"
 # XXX missing plugins: fakemetadata.
 makedepends="grilo-devel gom-devel json-glib-devel
@@ -17,7 +17,7 @@ maintainer="Orphaned <orphan@voidlinux.org>"
 license="LGPL-2.1-or-later"
 homepage="https://wiki.gnome.org/Projects/Grilo"
 distfiles="${GNOME_SITE}/grilo-plugins/${version%.*}/grilo-plugins-${version}.tar.xz"
-checksum=686844b34ec73b24931ff6cc4f6033f0072947a6db60acdc7fb3eaf157a581c8
+checksum=fe6f4dbe586c6b8ba2406394e202f22d009d642a96eb3a54f32f6a21d084cdcb
 make_check=no # feel free to fix
 
 post_patch() {

  parent reply	other threads:[~2024-02-29  5:14 UTC|newest]

Thread overview: 30+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2024-02-28  5:34 [PR PATCH] grilo: " chrysos349
2024-02-28  5:55 ` oreo639
2024-02-28  6:00 ` [PR PATCH] [Updated] " chrysos349
2024-02-28  6:01 ` grilo{,-plugins}: " chrysos349
2024-02-28  7:02 ` oreo639
2024-02-28  7:21 ` [PR PATCH] [Updated] " chrysos349
2024-02-28  7:31 ` [PR REVIEW] " oreo639
2024-02-28  7:38 ` chrysos349
2024-02-28  7:39 ` chrysos349
2024-02-28  7:44 ` oreo639
2024-02-28  7:47 ` oreo639
2024-02-28  7:48 ` oreo639
2024-02-28  7:49 ` oreo639
2024-02-28  7:51 ` oreo639
2024-02-28  7:54 ` oreo639
2024-02-28  7:54 ` oreo639
2024-02-28  7:55 ` oreo639
2024-02-28  8:01 ` oreo639
2024-02-28  8:05 ` oreo639
2024-02-28  9:01 ` [PR PATCH] [Updated] " chrysos349
2024-02-28 11:46 ` [PR REVIEW] " chrysos349
2024-02-28 22:32 ` oreo639
2024-02-28 22:32 ` oreo639
2024-02-28 22:35 ` oreo639
2024-02-28 22:36 ` oreo639
2024-02-28 22:37 ` oreo639
2024-02-28 22:37 ` oreo639
2024-02-29  5:14 ` chrysos349 [this message]
2024-02-29  5:15 ` chrysos349
2024-02-29  6:31 ` [PR PATCH] [Merged]: " oreo639

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20240229051425.AD52F24FC8@inbox.vuxu.org \
    --to=chrysos349@users.noreply.github.com \
    --cc=ml@inbox.vuxu.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
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 NNTP newsgroup(s).