diff options
author | Kurt Kanzenbach <kurt@kmk-computers.de> | 2021-04-24 12:18:01 +0200 |
---|---|---|
committer | Kurt Kanzenbach <kurt@kmk-computers.de> | 2021-04-24 12:18:01 +0200 |
commit | 33d6cebd06e2d9c041cdc48b242102edb1d852e8 (patch) | |
tree | 9c045cf65f222976751c3ea75bcdd1b5a64d8392 | |
parent | dev-libs/StringiFor: fix call gfortran directly, pass system LDFLAGS (diff) | |
download | guru-33d6cebd06e2d9c041cdc48b242102edb1d852e8.tar.gz guru-33d6cebd06e2d9c041cdc48b242102edb1d852e8.tar.bz2 guru-33d6cebd06e2d9c041cdc48b242102edb1d852e8.zip |
dev-util/xdp-tools: Don't call readelf directly
Allow to override it.
Closes: https://bugs.gentoo.org/785214
Package-Manager: Portage-3.0.17, Repoman-3.0.2
Signed-off-by: Kurt Kanzenbach <kurt@kmk-computers.de>
-rw-r--r-- | dev-util/xdp-tools/files/xdp-tools-1.1.0-readelf.patch | 52 | ||||
-rw-r--r-- | dev-util/xdp-tools/xdp-tools-1.1.0.ebuild | 6 |
2 files changed, 57 insertions, 1 deletions
diff --git a/dev-util/xdp-tools/files/xdp-tools-1.1.0-readelf.patch b/dev-util/xdp-tools/files/xdp-tools-1.1.0-readelf.patch new file mode 100644 index 0000000000..be68db3a8c --- /dev/null +++ b/dev-util/xdp-tools/files/xdp-tools-1.1.0-readelf.patch @@ -0,0 +1,52 @@ +From: Kurt Kanzenbach <kurt@kmk-computers.de> +Subject: make: Fix hardcoded readelf + +Don't hardcode readelf. Allow to reuse something else. + +Signed-off-by: Kurt Kanzenbach <kurt@kmk-computers.de> + +--- + lib/libxdp/Makefile | 10 +++++----- + 1 file changed, 5 insertions(+), 5 deletions(-) + +--- a/lib/libxdp/Makefile ++++ b/lib/libxdp/Makefile +@@ -26,7 +26,7 @@ TEMPLATED_SOURCES := xdp-dispatcher.c + + CFLAGS += -I$(HEADER_DIR) -I$(LIB_DIR)/util + BPF_CFLAGS += -I$(HEADER_DIR) +- ++READELF ?= readelf + + ifndef BUILD_STATIC_ONLY + SHARED_LIBS := $(OBJDIR)/libxdp.so \ +@@ -88,12 +88,12 @@ $(SHARED_OBJDIR)/%.o: %.c $(EXTRA_LIB_DE + + XDP_IN_SHARED := $(SHARED_OBJDIR)/libxdp.o + +-GLOBAL_SYM_COUNT = $(shell readelf -s --wide $(XDP_IN_SHARED) | \ ++GLOBAL_SYM_COUNT = $(shell $(READELF) -s --wide $(XDP_IN_SHARED) | \ + cut -d "@" -f1 | sed 's/_v[0-9]_[0-9]_[0-9].*//' | \ + sed 's/\[.*\]//' | \ + awk '/GLOBAL/ && /DEFAULT/ && !/UND/ {print $$NF}' | \ + sort -u | wc -l) +-VERSIONED_SYM_COUNT = $(shell readelf --dyn-syms --wide $(OBJDIR)/libxdp.so | \ ++VERSIONED_SYM_COUNT = $(shell $(READELF) --dyn-syms --wide $(OBJDIR)/libxdp.so | \ + grep -Eo '[^ ]+@LIBXDP_' | cut -d@ -f1 | sort -u | wc -l) + + check: check_abi +@@ -105,12 +105,12 @@ check_abi: $(OBJDIR)/libxdp.so + "versioned symbols in $^ ($(VERSIONED_SYM_COUNT))." \ + "Please make sure all symbols are" \ + "versioned in $(VERSION_SCRIPT)." >&2; \ +- readelf -s --wide $(XDP_IN_SHARED) | \ ++ $(READELF) -s --wide $(XDP_IN_SHARED) | \ + cut -d "@" -f1 | sed 's/_v[0-9]_[0-9]_[0-9].*//' | \ + sed 's/\[.*\]//' | \ + awk '/GLOBAL/ && /DEFAULT/ && !/UND/ {print $$NF}'| \ + sort -u > $(OUTPUT)libxdp_global_syms.tmp; \ +- readelf --dyn-syms --wide $(OUTPUT)libxdp.so | \ ++ $(READELF) --dyn-syms --wide $(OUTPUT)libxdp.so | \ + grep -Eo '[^ ]+@LIBXDP_' | cut -d@ -f1 | \ + sort -u > $(OUTPUT)libxdp_versioned_syms.tmp; \ + diff -u $(OUTPUT)libxdp_global_syms.tmp \ diff --git a/dev-util/xdp-tools/xdp-tools-1.1.0.ebuild b/dev-util/xdp-tools/xdp-tools-1.1.0.ebuild index 1f4477a5d7..71dd1e00e8 100644 --- a/dev-util/xdp-tools/xdp-tools-1.1.0.ebuild +++ b/dev-util/xdp-tools/xdp-tools-1.1.0.ebuild @@ -40,7 +40,10 @@ DEPEND="${RDEPEND} sys-devel/llvm:10 " -PATCHES=( "${FILESDIR}/${P}-install.patch" ) +PATCHES=( + "${FILESDIR}/${P}-install.patch" + "${FILESDIR}/${P}-readelf.patch" +) pkg_setup() { llvm_pkg_setup @@ -54,6 +57,7 @@ src_compile() { emake \ CC="$(tc-getCC)" \ AR="$(tc-getAR)" \ + READELF=$(tc-getREADELF) \ PRODUCTION=1 \ DYNAMIC_LIBXDP=1 \ FORCE_SYSTEM_LIBBPF=1 \ |