Nico Huber submitted this change.

View Change

Approvals: build bot (Jenkins): Verified Nico Huber: Looks good to me, approved
Makefile: use STRIP_ARGS = -s on all systems

Set -s (--strip-all) as STRIP_ARGS explicitly on all systems instead of
only on systems (SunOS / Solaris) where strip-all is not the default
behavior.

Change-Id: I2c80d553fb54d94bc65f07e700c90dd1bae4854e
Signed-off-by: Thomas Heijligen <thomas.heijligen@secunet.de>
Reviewed-on: https://review.coreboot.org/c/flashrom/+/58246
Reviewed-by: Nico Huber <nico.h@gmx.de>
Tested-by: build bot (Jenkins) <no-reply@coreboot.org>
---
M Makefile
1 file changed, 1 insertion(+), 4 deletions(-)

diff --git a/Makefile b/Makefile
index 8e06dee..29d1020 100644
--- a/Makefile
+++ b/Makefile
@@ -28,6 +28,7 @@
# Note for anyone editing this Makefile: gnumake will happily ignore any
# changes in this Makefile to variables set on the command line.
STRIP ?= strip
+STRIP_ARGS = -s
INSTALL = install
DIFF = diff
PREFIX ?= /usr/local
@@ -157,10 +158,6 @@
CC = gcc
endif

-ifneq ($(HOST_OS), SunOS)
-STRIP_ARGS = -s
-endif
-
# Determine the destination OS.
# IMPORTANT: The following line must be placed before TARGET_OS is ever used
# (of course), but should come after any lines setting CC because the line

2 is the latest approved patch-set. No files were changed between the latest approved patch-set and the submitted one.

To view, visit change 58246. To unsubscribe, or for help writing mail filters, visit settings.

Gerrit-Project: flashrom
Gerrit-Branch: master
Gerrit-Change-Id: I2c80d553fb54d94bc65f07e700c90dd1bae4854e
Gerrit-Change-Number: 58246
Gerrit-PatchSet: 4
Gerrit-Owner: Thomas Heijligen <src@posteo.de>
Gerrit-Reviewer: Felix Singer <felixsinger@posteo.net>
Gerrit-Reviewer: Angel Pons <th3fanbus@gmail.com>
Gerrit-Reviewer: Nico Huber <nico.h@gmx.de>
Gerrit-Reviewer: build bot (Jenkins) <no-reply@coreboot.org>
Gerrit-CC: Paul Menzel <paulepanter@mailbox.org>
Gerrit-MessageType: merged