[gentoo-commits] repo/gentoo:master commit in: media-sound/playerctl/files/, media-sound/playerctl/

[Göktürk Yüksek]

commit:     988c0ffb4c868eb82efc0a37169b96c3548dbd5e
Author:     Göktürk Yüksek <gokturk <AT> gentoo <DOT> org>
AuthorDate: Tue Apr  9 02:08:11 2019 +0000
Commit:     Göktürk Yüksek <gokturk <AT> gentoo <DOT> org>
CommitDate: Tue Apr  9 02:08:11 2019 +0000
URL:        https://gitweb.gentoo.org/repo/gentoo.git/commit/?id=988c0ffb

media-sound/playerctl: fix gdbus-codegen handling in configure.ac

In configure.ac, the **absolute** path of gdbus-codegen (as obtained
by pkg-config) is passed to AC_PATH_PROG. The macro then appends this
path to the directories in PATH and tests if the result is
executable. This internally results in checking for
'/usr/bin/usr/bin/gdbus-codegen'. The macro is redundant because the
value returned by pkg-config is sufficient.

Closes: https://bugs.gentoo.org/682788
Package-Manager: Portage-2.3.62, Repoman-2.3.12
Signed-off-by: Göktürk Yüksek <gokturk <AT> gentoo.org>

 .../playerctl/files/playerctl-gdbus-codegen-fix.patch     | 15 +++++++++++++++
 media-sound/playerctl/playerctl-0.5.0.ebuild              |  4 +++-
 media-sound/playerctl/playerctl-0.6.0.ebuild              |  4 +++-
 3 files changed, 21 insertions(+), 2 deletions(-)

diff --git a/media-sound/playerctl/files/playerctl-gdbus-codegen-fix.patch b/media-sound/playerctl/files/playerctl-gdbus-codegen-fix.patch
new file mode 100644
index 00000000000..06c073aa276
--- /dev/null
+++ b/media-sound/playerctl/files/playerctl-gdbus-codegen-fix.patch
@@ -0,0 +1,15 @@
+--- a/configure.ac
++++ b/configure.ac
+@@ -21,10 +21,11 @@
+ PKG_CHECK_MODULES([GOBJECT], [gobject-2.0 >= 2.38])
+ PKG_CHECK_MODULES([GIO], [gio-unix-2.0])
+ 
+-AC_PATH_PROG([GDBUS_CODEGEN], [`$PKG_CONFIG --variable=gdbus_codegen gio-2.0`])
++GDBUS_CODEGEN=`$PKG_CONFIG --variable=gdbus_codegen gio-2.0`
+ if test -z "$GDBUS_CODEGEN"; then
+     AC_MSG_ERROR([*** gdbus-codegen is required to build playerctl])
+ fi
++AC_SUBST(GDBUS_CODEGEN)
+ 
+ # Checks for typedefs, structures, and compiler characteristics
+ AC_PROG_CC_STDC

diff --git a/media-sound/playerctl/playerctl-0.5.0.ebuild b/media-sound/playerctl/playerctl-0.5.0.ebuild
index a06816f38bd..0a05e650f20 100644
--- a/media-sound/playerctl/playerctl-0.5.0.ebuild
+++ b/media-sound/playerctl/playerctl-0.5.0.ebuild
@@ -1,4 +1,4 @@
-# Copyright 1999-2017 Gentoo Foundation
+# Copyright 1999-2019 Gentoo Authors
 # Distributed under the terms of the GNU General Public License v2
 
 EAPI=6
@@ -29,6 +29,8 @@ src_prepare() {
 		echo 'EXTRA_DIST = ' > gtk-doc.make || die
 	fi
 
+	eapply "${FILESDIR}"/"${PN}"-gdbus-codegen-fix.patch
+
 	default
 	eautoreconf
 }

diff --git a/media-sound/playerctl/playerctl-0.6.0.ebuild b/media-sound/playerctl/playerctl-0.6.0.ebuild
index 8741ea6e29e..4effb5f74a6 100644
--- a/media-sound/playerctl/playerctl-0.6.0.ebuild
+++ b/media-sound/playerctl/playerctl-0.6.0.ebuild
@@ -1,4 +1,4 @@
-# Copyright 1999-2018 Gentoo Foundation
+# Copyright 1999-2019 Gentoo Authors
 # Distributed under the terms of the GNU General Public License v2
 
 EAPI=6
@@ -29,6 +29,8 @@ src_prepare() {
 		echo 'EXTRA_DIST = ' > gtk-doc.make || die
 	fi
 
+	eapply "${FILESDIR}"/"${PN}"-gdbus-codegen-fix.patch
+
 	default
 	eautoreconf
 }