public inbox for gentoo-commits@lists.gentoo.org
 help / color / mirror / Atom feed
From: "Göktürk Yüksek" <gokturk@gentoo.org>
To: gentoo-commits@lists.gentoo.org
Subject: [gentoo-commits] repo/gentoo:master commit in: media-sound/playerctl/files/, media-sound/playerctl/
Date: Tue,  9 Apr 2019 02:08:58 +0000 (UTC)	[thread overview]
Message-ID: <1554775691.988c0ffb4c868eb82efc0a37169b96c3548dbd5e.gokturk@gentoo> (raw)

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
 }


             reply	other threads:[~2019-04-09  2:09 UTC|newest]

Thread overview: 2+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2019-04-09  2:08 Göktürk Yüksek [this message]
  -- strict thread matches above, loose matches on Subject: below --
2019-06-13 23:21 [gentoo-commits] repo/gentoo:master commit in: media-sound/playerctl/files/, media-sound/playerctl/ Göktürk Yüksek

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=1554775691.988c0ffb4c868eb82efc0a37169b96c3548dbd5e.gokturk@gentoo \
    --to=gokturk@gentoo.org \
    --cc=gentoo-commits@lists.gentoo.org \
    --cc=gentoo-dev@lists.gentoo.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