[gentoo-commits] repo/gentoo:master commit in: dev-libs/opensc/

["Sam James" <sam@gentoo.org>]

commit:     66406184c8ab29ae1a03f4e3b9f8298208ea8f94
Author:     Matoro Mahri <matoro_gentoo <AT> matoro <DOT> tk>
AuthorDate: Wed Mar 13 18:29:26 2024 +0000
Commit:     Sam James <sam <AT> gentoo <DOT> org>
CommitDate: Fri Mar 15 04:07:10 2024 +0000
URL:        https://gitweb.gentoo.org/repo/gentoo.git/commit/?id=66406184

dev-libs/opensc: fix test search path for softhsm, disable broken

These tests are currently getting run only when dev-libs/softhsm is
installed, BUT on some architectures (including amd64), they use the
wrong search path (we install in /usr/$(get_libdir)/softhsm) and thus
think it's not installed anyway and skip.

Firstly, this commit enables these tests by overriding the search path.
However, one of the tests is broken (all architectures).  This seems to
be at least partially known upstream as there is a long trail of commits
disabling it for each individual release tag; but upstream seems to
recently have stopped disabling it for unknown reasons.  Given the
flakiness, disable this specific test downstream.

Bug: https://bugs.gentoo.org/909781
Signed-off-by: Matoro Mahri <matoro_gentoo <AT> matoro.tk>
Closes: https://github.com/gentoo/gentoo/pull/35737
Signed-off-by: Sam James <sam <AT> gentoo.org>

 ...{opensc-9999.ebuild => opensc-0.25.0-r1.ebuild} | 30 +++++++++++++---------
 dev-libs/opensc/opensc-9999.ebuild                 | 30 +++++++++++++---------
 2 files changed, 36 insertions(+), 24 deletions(-)

diff --git a/dev-libs/opensc/opensc-9999.ebuild b/dev-libs/opensc/opensc-0.25.0-r1.ebuild
similarity index 72%
copy from dev-libs/opensc/opensc-9999.ebuild
copy to dev-libs/opensc/opensc-0.25.0-r1.ebuild
index a470c4913c37..5546950912e9 100644
--- a/dev-libs/opensc/opensc-9999.ebuild
+++ b/dev-libs/opensc/opensc-0.25.0-r1.ebuild
@@ -1,23 +1,23 @@
-# Copyright 1999-2023 Gentoo Authors
+# Copyright 1999-2024 Gentoo Authors
 # Distributed under the terms of the GNU General Public License v2
 
 EAPI=8
 
-inherit bash-completion-r1 libtool
+inherit autotools bash-completion-r1
 
 DESCRIPTION="Libraries and applications to access smartcards"
 HOMEPAGE="https://github.com/OpenSC/OpenSC/wiki"
 
 if [[ ${PV} == *9999 ]]; then
-	inherit autotools git-r3
+	inherit git-r3
 	EGIT_REPO_URI="https://github.com/OpenSC/OpenSC.git"
 else
 	SRC_URI="https://github.com/OpenSC/OpenSC/releases/download/${PV}/${P}.tar.gz"
-	KEYWORDS="~amd64 ~ppc64 ~x86"
+	KEYWORDS="~amd64 ~arm ~arm64 ~hppa ~ia64 ~loong ~mips ~ppc ~ppc64 ~riscv ~s390 ~sparc ~x86"
 fi
 
 LICENSE="LGPL-2.1"
-SLOT="0"
+SLOT="0/11"
 IUSE="ctapi doc openct notify pace +pcsc-lite readline secure-messaging ssl test zlib"
 RESTRICT="!test? ( test )"
 
@@ -31,7 +31,10 @@ RDEPEND="zlib? ( sys-libs/zlib )
 DEPEND="${RDEPEND}
 	app-text/docbook-xsl-stylesheets
 	dev-libs/libxslt
-	test? ( dev-util/cmocka )"
+	test? (
+		dev-util/cmocka
+		dev-libs/softhsm
+	)"
 BDEPEND="virtual/pkgconfig"
 
 REQUIRED_USE="
@@ -41,13 +44,12 @@ REQUIRED_USE="
 	|| ( pcsc-lite openct ctapi )"
 
 src_prepare() {
+	# This test is known to fail, for a long time upstream has carried
+	# version-specific patches which they would update on every version bump.
+	# There doesn't appear to be a permanent solution yet.
+	sed -i "/test-pkcs11-tool-unwrap-wrap-test.sh/d" "tests/Makefile.am" || die
 	default
-
-	if [[ ${PV} == *9999 ]]; then
-		eautoreconf
-	else
-		elibtoolize
-	fi
+	eautoreconf
 }
 
 src_configure() {
@@ -71,6 +73,10 @@ src_configure() {
 		$(use_enable zlib)
 }
 
+src_test() {
+	P11LIB="/usr/$(get_libdir)/softhsm/libsofthsm2.so" default
+}
+
 src_install() {
 	default
 

diff --git a/dev-libs/opensc/opensc-9999.ebuild b/dev-libs/opensc/opensc-9999.ebuild
index a470c4913c37..5546950912e9 100644
--- a/dev-libs/opensc/opensc-9999.ebuild
+++ b/dev-libs/opensc/opensc-9999.ebuild
@@ -1,23 +1,23 @@
-# Copyright 1999-2023 Gentoo Authors
+# Copyright 1999-2024 Gentoo Authors
 # Distributed under the terms of the GNU General Public License v2
 
 EAPI=8
 
-inherit bash-completion-r1 libtool
+inherit autotools bash-completion-r1
 
 DESCRIPTION="Libraries and applications to access smartcards"
 HOMEPAGE="https://github.com/OpenSC/OpenSC/wiki"
 
 if [[ ${PV} == *9999 ]]; then
-	inherit autotools git-r3
+	inherit git-r3
 	EGIT_REPO_URI="https://github.com/OpenSC/OpenSC.git"
 else
 	SRC_URI="https://github.com/OpenSC/OpenSC/releases/download/${PV}/${P}.tar.gz"
-	KEYWORDS="~amd64 ~ppc64 ~x86"
+	KEYWORDS="~amd64 ~arm ~arm64 ~hppa ~ia64 ~loong ~mips ~ppc ~ppc64 ~riscv ~s390 ~sparc ~x86"
 fi
 
 LICENSE="LGPL-2.1"
-SLOT="0"
+SLOT="0/11"
 IUSE="ctapi doc openct notify pace +pcsc-lite readline secure-messaging ssl test zlib"
 RESTRICT="!test? ( test )"
 
@@ -31,7 +31,10 @@ RDEPEND="zlib? ( sys-libs/zlib )
 DEPEND="${RDEPEND}
 	app-text/docbook-xsl-stylesheets
 	dev-libs/libxslt
-	test? ( dev-util/cmocka )"
+	test? (
+		dev-util/cmocka
+		dev-libs/softhsm
+	)"
 BDEPEND="virtual/pkgconfig"
 
 REQUIRED_USE="
@@ -41,13 +44,12 @@ REQUIRED_USE="
 	|| ( pcsc-lite openct ctapi )"
 
 src_prepare() {
+	# This test is known to fail, for a long time upstream has carried
+	# version-specific patches which they would update on every version bump.
+	# There doesn't appear to be a permanent solution yet.
+	sed -i "/test-pkcs11-tool-unwrap-wrap-test.sh/d" "tests/Makefile.am" || die
 	default
-
-	if [[ ${PV} == *9999 ]]; then
-		eautoreconf
-	else
-		elibtoolize
-	fi
+	eautoreconf
 }
 
 src_configure() {
@@ -71,6 +73,10 @@ src_configure() {
 		$(use_enable zlib)
 }
 
+src_test() {
+	P11LIB="/usr/$(get_libdir)/softhsm/libsofthsm2.so" default
+}
+
 src_install() {
 	default