diff mbox series

[meta-oe,1/4] grpc: remove unneeded dependencies

Message ID 20250208150308.2451484-1-peter.marko@siemens.com
State Accepted
Headers show
Series [meta-oe,1/4] grpc: remove unneeded dependencies | expand

Commit Message

Peter Marko Feb. 8, 2025, 3:03 p.m. UTC 
From: Peter Marko <peter.marko@siemens.com>

googletest is needed only if we would build tests, and then it should be
conditional on PTEST_ENABLED

libnsl is not needed. there is no linking command in any cmake.
The only grep hit is in cares.cmake to disable it.
Commit 2ffc7b45d638a89535b963533d39b8b13e4c8c1e which added it talks
about host distros, but also such case would be wrong when adding it to
general depends (should be in native only depends) and it would have to
have be reference in cmake (in grpc or in one of recipe-sysroot).

Also sort the dependencies in the process.

Signed-off-by: Peter Marko <peter.marko@siemens.com>
---
 meta-oe/recipes-devtools/grpc/grpc_1.68.0.bb | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)
diff mbox series

Patch

diff --git a/meta-oe/recipes-devtools/grpc/grpc_1.68.0.bb b/meta-oe/recipes-devtools/grpc/grpc_1.68.0.bb
index caf355bb52..b82b802e3c 100644
--- a/meta-oe/recipes-devtools/grpc/grpc_1.68.0.bb
+++ b/meta-oe/recipes-devtools/grpc/grpc_1.68.0.bb
@@ -6,8 +6,8 @@  SECTION = "libs"
 LICENSE = "Apache-2.0 & BSD-3-Clause & MPL-2.0"
 LIC_FILES_CHKSUM = "file://LICENSE;md5=731e401b36f8077ae0c134b59be5c906"
 
-DEPENDS = "c-ares protobuf protobuf-native openssl libnsl2 abseil-cpp re2 zlib"
-DEPENDS:append:class-target = " googletest grpc-native "
+DEPENDS = "abseil-cpp c-ares openssl protobuf protobuf-native re2 zlib"
+DEPENDS:append:class-target = " grpc-native"
 DEPENDS:append:class-nativesdk = " grpc-native "
 
 PACKAGE_BEFORE_PN = "${PN}-compiler"