From 3090ed3f403cae8ec297aa8cbe8f7f7114c4dc75 Mon Sep 17 00:00:00 2001 From: sternenseemann Date: Fri, 29 Oct 2021 12:12:58 +0200 Subject: [PATCH] writers: use platform's default strip bintools points to the default bintools (e. g. cctools, binutils, llvmPackages.bintools) for the next stage. So instead of using GNU binutils' strip which may not support the current platform, we need to use buildPackages.bintools-unwrapped. Additionally we need to use `-S` over `--strip-unneeded` as the latter is a GNU binutils-specific flag and not supported by cctools. Also add the correct targetPrefix in order to support cross-compilation correctly here at least. --- pkgs/build-support/writers/default.nix | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/pkgs/build-support/writers/default.nix b/pkgs/build-support/writers/default.nix index 111ec68a6021..09ee7d738658 100644 --- a/pkgs/build-support/writers/default.nix +++ b/pkgs/build-support/writers/default.nix @@ -1,4 +1,4 @@ -{ pkgs, lib, gawk, gnused, gixy }: +{ pkgs, buildPackages, lib, gawk, gnused, gixy }: with lib; rec { @@ -77,7 +77,7 @@ rec { }) '' ${compileScript} ${lib.optionalString strip - "${pkgs.binutils-unwrapped}/bin/strip --strip-unneeded $out"} + "${lib.getBin buildPackages.bintools-unwrapped}/bin/${buildPackages.bintools-unwrapped.targetPrefix}strip -S $out"} ${optionalString (types.path.check nameOrPath) '' mv $out tmp mkdir -p $out/$(dirname "${nameOrPath}")