diff mbox series

[meta-darwin] clang: remove runtime dependency to perl

Message ID 20230828161407.2607186-1-ecordonnier@snap.com
State New
Headers show
Series [meta-darwin] clang: remove runtime dependency to perl | expand

Commit Message

Etienne Cordonnier Aug. 28, 2023, 4:14 p.m. UTC 
From: Etienne Cordonnier <ecordonnier@snap.com>

Commit https://github.com/kraj/meta-clang/commit/68ec449f97ffa58d in meta-clang adds a dependency to perl,
however at the moment perl does not cross-compile for darwin (the build tries to use readelf which does not
exist on darwin, instead of using objdump).

This dependency is needed only for optional runtime tools, so just remove it at the moment.

Signed-off-by: Etienne Cordonnier <ecordonnier@snap.com>
---
 recipes-devtools/clang/clang_%.bbappend | 14 ++++++++++++++
 1 file changed, 14 insertions(+)
diff mbox series

Patch

diff --git a/recipes-devtools/clang/clang_%.bbappend b/recipes-devtools/clang/clang_%.bbappend
index 25a9bf7..ce2a3b3 100644
--- a/recipes-devtools/clang/clang_%.bbappend
+++ b/recipes-devtools/clang/clang_%.bbappend
@@ -8,6 +8,20 @@  PACKAGECONFIG:remove:class-nativesdk:darwin21 = "shared-libs"
 DEPENDS:remove:class-nativesdk = "clang-crosssdk-${SDK_ARCH}"
 DEPENDS:append:class-nativesdk = " clang-crosssdk-${SDK_SYS}"
 
+# perl tries to call readelf, which does not exist on darwin (it would need
+# to call objdump instead but the detection logic does not work for some reason)
+RDEPENDS:${PN}:remove:class-nativesdk:darwin21 = " \
+    perl-module-digest-md5 \
+    perl-module-file-basename \
+    perl-module-file-copy \
+    perl-module-file-find \
+    perl-module-file-path \
+    perl-module-findbin \
+    perl-module-hash-util \
+    perl-module-sys-hostname \
+    perl-module-term-ansicolor \
+"
+
 COMPILER_RT:class-nativesdk:toolchain-clang:runtime-llvm:darwin21 = ""
 LIBCPLUSPLUS:class-nativesdk:toolchain-clang:darwin21 = " -stdlib=libstdc++"