public inbox for gentoo-commits@lists.gentoo.org
 help / color / mirror / Atom feed
From: "Marek Szuba" <marecki@gentoo.org>
To: gentoo-commits@lists.gentoo.org
Subject: [gentoo-commits] repo/gentoo:master commit in: x11-themes/gtk-engines/
Date: Tue, 22 Dec 2020 17:32:33 +0000 (UTC)	[thread overview]
Message-ID: <1608658337.205b0f865a75f324a828a7c8725f8893b76cecfb.marecki@gentoo> (raw)

commit:     205b0f865a75f324a828a7c8725f8893b76cecfb
Author:     Marek Szuba <marecki <AT> gentoo <DOT> org>
AuthorDate: Tue Dec 22 17:20:32 2020 +0000
Commit:     Marek Szuba <marecki <AT> gentoo <DOT> org>
CommitDate: Tue Dec 22 17:32:17 2020 +0000
URL:        https://gitweb.gentoo.org/repo/gentoo.git/commit/?id=205b0f86

x11-themes/gtk-engines-2.20.2-r100: make USE=lua work for non-native ABIs

As predicted, I have ended up simply patching configure.

Closes: https://bugs.gentoo.org/752888
Signed-off-by: Marek Szuba <marecki <AT> gentoo.org>

 .../gtk-engines/gtk-engines-2.20.2-r100.ebuild       | 20 +++++++-------------
 1 file changed, 7 insertions(+), 13 deletions(-)

diff --git a/x11-themes/gtk-engines/gtk-engines-2.20.2-r100.ebuild b/x11-themes/gtk-engines/gtk-engines-2.20.2-r100.ebuild
index 597dc742bc9..39c7e74458e 100644
--- a/x11-themes/gtk-engines/gtk-engines-2.20.2-r100.ebuild
+++ b/x11-themes/gtk-engines/gtk-engines-2.20.2-r100.ebuild
@@ -42,25 +42,19 @@ PATCHES=(
 
 src_prepare() {
 	gnome2_src_prepare
+	# pkgconfig wrapper set up by lua-single.eclass is not multilib-compatible
+	# at present so point Autoconf directly to the correct implementation.
+	# We patch configure rather than configure.ac because running 'eautoreconf'
+	# results for some reason in corrupted test Makefiles.
+	sed -i -e "s|\"lua\"|\"${ELUA}\"|g" configure || die
 }
 
 multilib_src_configure() {
 	local confopts=(
 		--enable-animation
+		$(use_enable lua)
+		$(use_with lua system-lua)
 	)
-	# TODO: fix system-lua detection so that it works for non-native ABIs,
-	# native builds rely on the pkgconfig wrapper set up by lua-single.eclass
-	# but that wrapper is not multilib-compatible.
-	if multilib_is_native_abi; then
-		confopts+=(
-			$(use_enable lua)
-			$(use_with lua system-lua)
-		)
-	else
-		confopts+=(
-			--disable-lua
-		)
-	fi
 	ECONF_SOURCE=${S} gnome2_src_configure "${confopts[@]}"
 }
 


             reply	other threads:[~2020-12-22 17:32 UTC|newest]

Thread overview: 24+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2020-12-22 17:32 Marek Szuba [this message]
  -- strict thread matches above, loose matches on Subject: below --
2023-12-07 16:25 [gentoo-commits] repo/gentoo:master commit in: x11-themes/gtk-engines/ Arthur Zamarin
2022-08-30 17:50 Sam James
2022-06-28  2:03 WANG Xuerui
2021-01-30 11:58 Sam James
2021-01-24 11:14 Lars Wendler
2021-01-24  3:52 Sam James
2021-01-23 22:51 Sam James
2021-01-23 22:42 Sam James
2021-01-23  4:20 Sam James
2021-01-23  4:09 Sam James
2021-01-06 12:07 Fabian Groffen
2020-02-09 12:27 Mart Raudsepp
2017-08-26 10:07 Gilles Dartiguelongue
2017-08-26 10:07 Gilles Dartiguelongue
2015-11-16 14:08 Agostino Sarubbo
2015-11-12 11:07 Agostino Sarubbo
2015-10-18 11:29 Mikle Kolyada
2015-10-03  8:48 Markus Meier
2015-09-28  4:44 Jeroen Roovers
2015-09-20  6:11 Jeroen Roovers
2015-09-16  8:43 Tobias Klausmann
2015-09-12  9:11 Agostino Sarubbo
2015-09-07 21:55 Mikle Kolyada

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=1608658337.205b0f865a75f324a828a7c8725f8893b76cecfb.marecki@gentoo \
    --to=marecki@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