public inbox for gentoo-commits@lists.gentoo.org
 help / color / mirror / Atom feed
From: "Golubev Alexander" <fatzer2@gmail.com>
To: gentoo-commits@lists.gentoo.org
Subject: [gentoo-commits] repo/proj/guru:dev commit in: net-libs/usockets/
Date: Wed, 13 Nov 2024 22:57:07 +0000 (UTC)	[thread overview]
Message-ID: <1731538598.97f47c39bfa2f995da9557c696ce80a5df88aef1.fatzer2@gentoo> (raw)

commit:     97f47c39bfa2f995da9557c696ce80a5df88aef1
Author:     Alexander Golubev <fatzer2 <AT> gmail <DOT> com>
AuthorDate: Wed Nov 13 22:53:29 2024 +0000
Commit:     Golubev Alexander <fatzer2 <AT> gmail <DOT> com>
CommitDate: Wed Nov 13 22:56:38 2024 +0000
URL:        https://gitweb.gentoo.org/repo/proj/guru.git/commit/?id=97f47c39

net-libs/usockets: update EAPI 7 -> 8

Also:
- avoid downloading submodules for live (boringssl is quite bulky)
- fix test

Closes: https://bugs.gentoo.org/820296
Signed-off-by: Alexander Golubev <fatzer2 <AT> gmail.com>

 net-libs/usockets/usockets-0.8.8.ebuild | 32 +++++++++++++++++++++++---------
 net-libs/usockets/usockets-9999.ebuild  | 32 +++++++++++++++++++++++---------
 2 files changed, 46 insertions(+), 18 deletions(-)

diff --git a/net-libs/usockets/usockets-0.8.8.ebuild b/net-libs/usockets/usockets-0.8.8.ebuild
index 328d32ea9..93f82656b 100644
--- a/net-libs/usockets/usockets-0.8.8.ebuild
+++ b/net-libs/usockets/usockets-0.8.8.ebuild
@@ -1,7 +1,7 @@
-# Copyright 2019-2021 Gentoo Authors
+# Copyright 2019-2024 Gentoo Authors
 # Distributed under the terms of the GNU General Public License v2
 
-EAPI=7
+EAPI=8
 
 inherit toolchain-funcs
 
@@ -11,6 +11,7 @@ HOMEPAGE="https://github.com/uNetworking/uSockets"
 if [[ ${PV} == 9999 ]]; then
 	inherit git-r3
 	EGIT_REPO_URI="https://github.com/uNetworking/uSockets.git"
+	EGIT_SUBMODULES=( '-*' ) # We don't use any of bundled libraries from submodules
 else
 	SRC_URI="https://github.com/uNetworking/uSockets/archive/v${PV}.tar.gz -> ${P}.tar.gz"
 	KEYWORDS="~amd64 ~arm64 ~x86"
@@ -19,9 +20,10 @@ fi
 
 LICENSE="Apache-2.0"
 SLOT="0"
-IUSE="asio libuv +ssl test"
-REQUIRED_USE="?? ( asio libuv ) test? ( ssl )"
-RESTRICT="!test? ( test )"
+IUSE="asio libuv +ssl"
+REQUIRED_USE="?? ( asio libuv )"
+# Our only half-assed test requires ssl
+RESTRICT="!ssl? ( test )"
 
 DEPEND="
 	asio? ( dev-cpp/asio:= )
@@ -40,13 +42,25 @@ PATCHES=(
 src_configure() {
 	tc-export CC CXX AR
 	export VERSION="${PV%_*}" \
-	       LIB="$(get_libdir)" \
-	       WITH_OPENSSL="$(usex ssl 1 0)" \
-	       WITH_LIBUV="$(usex libuv 1 0)" \
-	       WITH_ASIO="$(usex asio 1 0)"
+		LIB="$(get_libdir)" \
+		WITH_OPENSSL="$(usex ssl 1 0)" \
+		WITH_LIBUV="$(usex libuv 1 0)" \
+		WITH_ASIO="$(usex asio 1 0)"
 	default
 }
 
+src_test() {
+	local saved_ulimit=$(ulimit -n)
+	# see https://bugs.gentoo.org/820296
+	if ! ulimit -n 10240; then
+		ewarn "Failed to set ulimit; ${PN} require ulimit -n 10240 to reliably pass tests"
+		ulimit -n 2048 || die "${PN} requires ulimit -n set to at least 2048 for tests"
+	fi
+
+	emake test
+	ulimit -n "${saved_ulimit}" || die "Failed restore ulimit to its original value"
+}
+
 src_install() {
 	default
 	einstalldocs

diff --git a/net-libs/usockets/usockets-9999.ebuild b/net-libs/usockets/usockets-9999.ebuild
index 328d32ea9..93f82656b 100644
--- a/net-libs/usockets/usockets-9999.ebuild
+++ b/net-libs/usockets/usockets-9999.ebuild
@@ -1,7 +1,7 @@
-# Copyright 2019-2021 Gentoo Authors
+# Copyright 2019-2024 Gentoo Authors
 # Distributed under the terms of the GNU General Public License v2
 
-EAPI=7
+EAPI=8
 
 inherit toolchain-funcs
 
@@ -11,6 +11,7 @@ HOMEPAGE="https://github.com/uNetworking/uSockets"
 if [[ ${PV} == 9999 ]]; then
 	inherit git-r3
 	EGIT_REPO_URI="https://github.com/uNetworking/uSockets.git"
+	EGIT_SUBMODULES=( '-*' ) # We don't use any of bundled libraries from submodules
 else
 	SRC_URI="https://github.com/uNetworking/uSockets/archive/v${PV}.tar.gz -> ${P}.tar.gz"
 	KEYWORDS="~amd64 ~arm64 ~x86"
@@ -19,9 +20,10 @@ fi
 
 LICENSE="Apache-2.0"
 SLOT="0"
-IUSE="asio libuv +ssl test"
-REQUIRED_USE="?? ( asio libuv ) test? ( ssl )"
-RESTRICT="!test? ( test )"
+IUSE="asio libuv +ssl"
+REQUIRED_USE="?? ( asio libuv )"
+# Our only half-assed test requires ssl
+RESTRICT="!ssl? ( test )"
 
 DEPEND="
 	asio? ( dev-cpp/asio:= )
@@ -40,13 +42,25 @@ PATCHES=(
 src_configure() {
 	tc-export CC CXX AR
 	export VERSION="${PV%_*}" \
-	       LIB="$(get_libdir)" \
-	       WITH_OPENSSL="$(usex ssl 1 0)" \
-	       WITH_LIBUV="$(usex libuv 1 0)" \
-	       WITH_ASIO="$(usex asio 1 0)"
+		LIB="$(get_libdir)" \
+		WITH_OPENSSL="$(usex ssl 1 0)" \
+		WITH_LIBUV="$(usex libuv 1 0)" \
+		WITH_ASIO="$(usex asio 1 0)"
 	default
 }
 
+src_test() {
+	local saved_ulimit=$(ulimit -n)
+	# see https://bugs.gentoo.org/820296
+	if ! ulimit -n 10240; then
+		ewarn "Failed to set ulimit; ${PN} require ulimit -n 10240 to reliably pass tests"
+		ulimit -n 2048 || die "${PN} requires ulimit -n set to at least 2048 for tests"
+	fi
+
+	emake test
+	ulimit -n "${saved_ulimit}" || die "Failed restore ulimit to its original value"
+}
+
 src_install() {
 	default
 	einstalldocs


             reply	other threads:[~2024-11-13 22:57 UTC|newest]

Thread overview: 21+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2024-11-13 22:57 Golubev Alexander [this message]
  -- strict thread matches above, loose matches on Subject: below --
2024-05-16 16:41 [gentoo-commits] repo/proj/guru:dev commit in: net-libs/usockets/ Aisha Tammy
2024-05-02 15:42 [gentoo-commits] repo/proj/guru:master " David Roman
2024-05-01  1:46 ` [gentoo-commits] repo/proj/guru:dev " David Roman
2024-04-30 23:55 Aisha Tammy
2021-09-21  9:52 Arthur Zamarin
2021-06-09  3:22 Theo Anderson
2021-05-03  9:52 Andrew Ammerlaan
2021-04-23 10:12 Aisha Tammy
2021-04-10 22:13 Aisha Tammy
2021-04-10 22:13 Aisha Tammy
2021-01-29 11:04 Aisha Tammy
2021-01-29 11:04 Aisha Tammy
2021-01-29  2:20 Aisha Tammy
2021-01-29  2:20 Aisha Tammy
2020-11-25 23:54 Aisha Tammy
2020-11-23 14:51 Aisha Tammy
2020-09-08 20:33 Aisha Tammy
2020-07-29 12:44 Aisha Tammy
2020-06-26 10:32 Aisha Tammy
2020-06-24  2:19 Aisha Tammy
2020-06-24  2:19 Aisha Tammy

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=1731538598.97f47c39bfa2f995da9557c696ce80a5df88aef1.fatzer2@gentoo \
    --to=fatzer2@gmail.com \
    --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