public inbox for gentoo-commits@lists.gentoo.org
 help / color / mirror / Atom feed
From: "Florian Schmaus" <flow@gentoo.org>
To: gentoo-commits@lists.gentoo.org
Subject: [gentoo-commits] repo/proj/guru:master commit in: app-editors/imhex/
Date: Wed, 26 Apr 2023 09:44:03 +0000 (UTC)	[thread overview]
Message-ID: <1682371369.c858e783d282e2b770996c2569ffaad1f5ca5556.flow@gentoo> (raw)

commit:     c858e783d282e2b770996c2569ffaad1f5ca5556
Author:     Henri Gasc <gasc <AT> eurecom <DOT> fr>
AuthorDate: Mon Apr 24 21:21:54 2023 +0000
Commit:     Florian Schmaus <flow <AT> gentoo <DOT> org>
CommitDate: Mon Apr 24 21:22:49 2023 +0000
URL:        https://gitweb.gentoo.org/repo/proj/guru.git/commit/?id=c858e783

app-editors/imhex: use CFLAGS and CXXFLAGS, disable testing

Closes: https://bugs.gentoo.org/904953
Closes: https://bugs.gentoo.org/904954
Signed-off-by: Henri Gasc <gasc <AT> eurecom.fr>

 app-editors/imhex/imhex-1.28.0.ebuild | 23 +++++++++--------------
 1 file changed, 9 insertions(+), 14 deletions(-)

diff --git a/app-editors/imhex/imhex-1.28.0.ebuild b/app-editors/imhex/imhex-1.28.0.ebuild
index 8f91766d1..be29cd82b 100644
--- a/app-editors/imhex/imhex-1.28.0.ebuild
+++ b/app-editors/imhex/imhex-1.28.0.ebuild
@@ -55,12 +55,14 @@ pkg_pretend() {
 
 src_prepare() {
 	default
-	# Due to network sandboxing, we can't do network test here.
+	# We are removing all tests altogether
+	# The tests need ImHex installed to succeed (see https://bugs.gentoo.org/attachment.cgi?id=860683), so we remove them
+	# We could use `ln "${BUILD_DIR}/lib/libimhex.so.${PV}" "/lib64/libimhex.so.${PV}"` and  circumvent sandboxing
 	sed -i \
-		-e 's/StoreAPI$/#StoreAPI/' \
-		-e 's/TipsAPI$/#TipsAPI/' \
-		-e 's/ContentAPI$/#ContentAPI/' \
-		"${S}/tests/helpers/CMakeLists.txt" || die
+		-e 's/enable_testing/#enable_testing/' \
+		-e 's/add_subdirectory(tests/#add_subdirectory(tests/' \
+		"${S}/CMakeLists.txt" || die
+
 	cmake_src_prepare
 }
 
@@ -69,8 +71,8 @@ src_configure() {
 		-D CMAKE_BUILD_TYPE="Release" \
 		-D CMAKE_C_COMPILER_LAUNCHER=ccache \
 		-D CMAKE_CXX_COMPILER_LAUNCHER=ccache \
-		-D CMAKE_C_FLAGS="-fuse-ld=lld" \
-		-D CMAKE_CXX_FLAGS="-fuse-ld=lld" \
+		-D CMAKE_C_FLAGS="-fuse-ld=lld ${CFLAGS}" \
+		-D CMAKE_CXX_FLAGS="-fuse-ld=lld ${CXXFLAGS}" \
 		-D CMAKE_OBJC_COMPILER_LAUNCHER=ccache \
 		-D CMAKE_OBJCXX_COMPILER_LAUNCHER=ccache \
 		-D CMAKE_SKIP_RPATH=ON \
@@ -97,13 +99,6 @@ src_configure() {
 	cmake_src_configure
 }
 
-src_test() {
-	pushd "${BUILD_DIR}" || die
-	emake unit_tests
-	popd || die
-	cmake_src_test
-}
-
 src_install() {
 	cmake_src_install
 


             reply	other threads:[~2023-04-26  9:44 UTC|newest]

Thread overview: 17+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2023-04-26  9:44 Florian Schmaus [this message]
  -- strict thread matches above, loose matches on Subject: below --
2024-08-31 13:06 [gentoo-commits] repo/proj/guru:master commit in: app-editors/imhex/ David Roman
2024-04-11 16:22 Julien Roy
2024-03-11 17:31 David Roman
2023-06-26 22:01 Haelwenn Monnier
2023-04-23  8:40 Florian Schmaus
2023-04-19  7:32 Florian Schmaus
2023-01-10 13:59 Florian Schmaus
2022-09-26  9:48 Andrew Ammerlaan
2022-09-08  9:31 Andrew Ammerlaan
2022-09-08  9:31 Andrew Ammerlaan
2022-09-08  9:31 Andrew Ammerlaan
2022-08-24 15:52 Ronny Gutbrod
2022-05-02  7:17 Florian Schmaus
2021-10-06  1:13 Theo Anderson
2021-07-07 14:02 Florian Schmaus
2021-07-05 15:48 Haelwenn Monnier

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=1682371369.c858e783d282e2b770996c2569ffaad1f5ca5556.flow@gentoo \
    --to=flow@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