public inbox for gentoo-commits@lists.gentoo.org
 help / color / mirror / Atom feed
From: "Sam James" <sam@gentoo.org>
To: gentoo-commits@lists.gentoo.org
Subject: [gentoo-commits] repo/gentoo:master commit in: net-libs/grpc/
Date: Mon, 21 Aug 2023 15:23:30 +0000 (UTC)	[thread overview]
Message-ID: <1692631401.61d0b43d739b703ee9f25abefb04ba0bdf1ac158.sam@gentoo> (raw)

commit:     61d0b43d739b703ee9f25abefb04ba0bdf1ac158
Author:     Sam James <sam <AT> gentoo <DOT> org>
AuthorDate: Mon Aug 21 15:03:41 2023 +0000
Commit:     Sam James <sam <AT> gentoo <DOT> org>
CommitDate: Mon Aug 21 15:23:21 2023 +0000
URL:        https://gitweb.gentoo.org/repo/gentoo.git/commit/?id=61d0b43d

net-libs/grpc: bundle abseil-cpp

abseil isn't *really* intended to be a system library and having a year-old
pinned dep like this causes havoc for other packages and leads to uninstallable
combinations (e.g. libreoffice-bin + anything needing grpc).

Please see also the extensive discussion in the linked PRs wrt abseil now
depending on re2 and the ABI implications of the used C++ std. versions with
abseil. It's not worth it and there's more downsides to unbundling than not.

Note that we can't rebundle re2 here because the build system unconditionally
then tries to install it (no toggle unlike for abseil-cpp).

Bug: https://bugs.gentoo.org/906811
Bug: https://github.com/gentoo/gentoo/pull/31388
Bug: https://github.com/gentoo/gentoo/pull/31609
Bug: https://github.com/gentoo/gentoo/pull/31893
Bug: https://github.com/gentoo/gentoo/pull/32281
Closes: https://bugs.gentoo.org/908373
Signed-off-by: Sam James <sam <AT> gentoo.org>

 net-libs/grpc/Manifest                                  |  1 +
 .../grpc/{grpc-1.52.1.ebuild => grpc-1.52.1-r1.ebuild}  | 17 +++++++++++++----
 2 files changed, 14 insertions(+), 4 deletions(-)

diff --git a/net-libs/grpc/Manifest b/net-libs/grpc/Manifest
index 748c6d78954c..f96b5140be56 100644
--- a/net-libs/grpc/Manifest
+++ b/net-libs/grpc/Manifest
@@ -1 +1,2 @@
+DIST abseil-cpp-20220623.1.tar.gz 1957483 BLAKE2B 5bcf60b5a55882bcb78c7095a9892f6d1e5819fb116e325a8299fb22780ab009f0e1d92d5d8aca5ae0d4acc6d60e620e3357a24e389059f6e75d94f1cbdc1ced SHA512 ab4fccd9a2bfa0c5ad4b56c8e8f8b7ec7a8eca8b6cc6959802acadd1da785e1feb078c6ac621808cd699c82717a9e637dc426d94b70a8db7f2a807059d41cbc2
 DIST grpc-1.52.1.tar.gz 8686340 BLAKE2B 0076ddd4a2840892f4ad11885e18f782b6a645337224974522c5f828d0226d6a1e19aaaf0f30ea3225d56118e0cfc251d45b8d5e6811c2d1a84e274ff037bb53 SHA512 06c69fb817af75b2610761a3a193178b749755eb7bed58875aa251def7c0c253cdaf02cf834c31c8b2cae7b01a6081e2aece4b131a162f64bd45ff0aff4d7758

diff --git a/net-libs/grpc/grpc-1.52.1.ebuild b/net-libs/grpc/grpc-1.52.1-r1.ebuild
similarity index 82%
rename from net-libs/grpc/grpc-1.52.1.ebuild
rename to net-libs/grpc/grpc-1.52.1-r1.ebuild
index c36f6dbeb491..780e0ab91b8f 100644
--- a/net-libs/grpc/grpc-1.52.1.ebuild
+++ b/net-libs/grpc/grpc-1.52.1-r1.ebuild
@@ -3,13 +3,20 @@
 
 EAPI=8
 
-inherit cmake
+inherit cmake flag-o-matic
 
 MY_PV="${PV//_pre/-pre}"
+# Check third_party/abseil-cpp's commit for each tag of ${PN}
+# It may correspond to a commit of abseil-cpp - if so, use the tarball for it
+# and not an arbitrary commit.
+ABSEIL_COMMIT="20220623.1"
 
 DESCRIPTION="Modern open source high performance RPC framework"
 HOMEPAGE="https://www.grpc.io"
-SRC_URI="https://github.com/${PN}/${PN}/archive/v${MY_PV}.tar.gz -> ${P}.tar.gz"
+SRC_URI="
+	https://github.com/${PN}/${PN}/archive/v${MY_PV}.tar.gz -> ${P}.tar.gz
+	https://github.com/abseil/abseil-cpp/archive/refs/tags/${ABSEIL_COMMIT}.tar.gz -> abseil-cpp-${ABSEIL_COMMIT}.tar.gz
+"
 
 LICENSE="Apache-2.0"
 # format is 0/${CORE_SOVERSION//./}.${CPP_SOVERSION//./} , check top level CMakeLists.txt
@@ -19,7 +26,6 @@ IUSE="doc examples test"
 
 # look for submodule versions in third_party dir
 RDEPEND="
-	=dev-cpp/abseil-cpp-20220623.1*:=
 	>=dev-libs/re2-0.2021.11.01:=
 	>=dev-libs/openssl-1.1.1:0=[-bindist(-)]
 	>=dev-libs/protobuf-3.18.1:=
@@ -72,7 +78,10 @@ src_configure() {
 
 	local mycmakeargs=(
 		-DgRPC_INSTALL=ON
-		-DgRPC_ABSL_PROVIDER=package
+		-DgRPC_ABSL_PROVIDER=module
+		-DgRPC_INSTALL=OFF
+		-DABSL_ENABLE_INSTALL=OFF
+		-DABSL_ROOT_DIR="${WORKDIR}"/abseil-cpp-${ABSEIL_COMMIT}
 		-DgRPC_BACKWARDS_COMPATIBILITY_MODE=OFF
 		-DgRPC_CARES_PROVIDER=package
 		-DgRPC_INSTALL_CMAKEDIR="$(get_libdir)/cmake/${PN}"


             reply	other threads:[~2023-08-21 15:23 UTC|newest]

Thread overview: 86+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2023-08-21 15:23 Sam James [this message]
  -- strict thread matches above, loose matches on Subject: below --
2024-09-17 16:00 [gentoo-commits] repo/gentoo:master commit in: net-libs/grpc/ Sam James
2024-09-12 21:59 Sam James
2024-09-04  9:32 Sam James
2024-07-09 12:52 Sam James
2023-12-25 19:22 Conrad Kostecki
2023-08-21 18:22 Sam James
2023-08-21 17:31 Sam James
2023-08-21 17:31 Sam James
2023-08-21 17:31 Sam James
2023-08-21 16:35 Sam James
2023-05-26 20:21 Andreas Sturmlechner
2023-05-14  8:48 Sam James
2023-05-14  8:48 Sam James
2023-05-14  8:48 Sam James
2023-02-27  9:10 WANG Xuerui
2023-02-24 17:11 Arthur Zamarin
2023-02-20 21:29 Arthur Zamarin
2023-02-16  5:52 WANG Xuerui
2023-02-12 18:42 Jakov Smolić
2023-02-12 18:42 Jakov Smolić
2023-02-12 18:36 Jakov Smolić
2023-02-11  7:47 WANG Xuerui
2023-02-11  7:47 WANG Xuerui
2023-02-11  7:47 WANG Xuerui
2023-02-07  0:35 Georgy Yakovlev
2022-12-24 14:49 Andreas Sturmlechner
2022-12-20 23:57 Sam James
2022-10-27 23:46 Sam James
2022-10-20 10:52 Jakov Smolić
2022-08-17 21:13 Sam James
2021-12-20 12:09 Agostino Sarubbo
2021-12-20  9:20 Arthur Zamarin
2021-12-20  5:34 Georgy Yakovlev
2021-12-20  5:31 Georgy Yakovlev
2021-12-20  5:10 Georgy Yakovlev
2021-12-20  5:04 Georgy Yakovlev
2021-11-14  1:59 Georgy Yakovlev
2021-06-21 23:31 Georgy Yakovlev
2021-06-21 23:31 Georgy Yakovlev
2021-05-30  1:43 Georgy Yakovlev
2021-05-15  2:21 Sam James
2021-05-02 10:16 Mikle Kolyada
2021-04-30 18:11 Georgy Yakovlev
2021-04-29 23:35 Georgy Yakovlev
2021-04-29 11:46 Michał Górny
2021-04-29  8:29 Georgy Yakovlev
2020-04-24 20:02 Georgy Yakovlev
2020-04-23 16:27 Stefan Strogin
2020-04-20  7:28 Georgy Yakovlev
2020-02-22  1:30 Georgy Yakovlev
2020-01-17 10:22 Georgy Yakovlev
2020-01-17 10:22 Georgy Yakovlev
2019-12-22  3:28 Georgy Yakovlev
2019-10-26  0:11 Georgy Yakovlev
2019-08-17  3:00 Georgy Yakovlev
2019-08-17  3:00 Georgy Yakovlev
2019-08-17  3:00 Georgy Yakovlev
2019-08-17  3:00 Georgy Yakovlev
2019-07-27  9:07 Georgy Yakovlev
2019-07-27  9:07 Georgy Yakovlev
2019-07-27  9:07 Georgy Yakovlev
2019-07-27  9:07 Georgy Yakovlev
2019-07-27  9:07 Georgy Yakovlev
2019-07-27  9:07 Georgy Yakovlev
2019-05-31 19:26 Georgy Yakovlev
2019-05-28  6:49 Georgy Yakovlev
2019-05-07 19:00 Georgy Yakovlev
2019-05-07 19:00 Georgy Yakovlev
2019-04-30  4:42 Georgy Yakovlev
2019-02-27  4:20 Georgy Yakovlev
2019-01-16  0:52 Georgy Yakovlev
2019-01-16  0:52 Georgy Yakovlev
2018-12-15  6:00 Georgy Yakovlev
2018-12-10  4:50 Georgy Yakovlev
2018-11-15  3:21 Georgy Yakovlev
2018-11-15  3:21 Georgy Yakovlev
2018-07-16 19:32 Georgy Yakovlev
2018-07-15 22:20 Georgy Yakovlev
2018-07-15 22:20 Georgy Yakovlev
2018-07-15 22:20 Georgy Yakovlev
2018-07-08 15:29 Jason Zaman
2018-06-17  1:18 Georgy Yakovlev
2018-05-30 13:24 Jason Zaman
2018-05-30 13:24 Jason Zaman
2018-05-03  8:59 Jason Zaman

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=1692631401.61d0b43d739b703ee9f25abefb04ba0bdf1ac158.sam@gentoo \
    --to=sam@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