Anastasia Klimchuk submitted this change.

View Change



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

Approvals: build bot (Jenkins): Verified Paul Menzel: Looks good to me, but someone else must approve Anastasia Klimchuk: Looks good to me, approved
Makefile: Use printf instead of echo -n

On MacOS `echo -n` is not working.

Change-Id: I96bb6c60e74133bbc86d5069cb076ee43f4bca83
Signed-off-by: Thomas Heijligen <thomas.heijligen@secunet.com>
Reviewed-on: https://review.coreboot.org/c/flashrom/+/74522
Reviewed-by: Paul Menzel <paulepanter@mailbox.org>
Reviewed-by: Anastasia Klimchuk <aklm@chromium.org>
Tested-by: build bot (Jenkins) <no-reply@coreboot.org>
---
M Makefile
1 file changed, 17 insertions(+), 1 deletion(-)

diff --git a/Makefile b/Makefile
index 0e16942..3258ac3 100644
--- a/Makefile
+++ b/Makefile
@@ -965,7 +965,7 @@

config:
@echo Building flashrom version $(VERSION)
- @echo -n "C compiler found: "
+ @printf "C compiler found: "
@if [ $(CC_WORKING) = yes ]; \
then $(CC) --version 2>/dev/null | head -1; \
else echo no; echo Aborting.; exit 1; fi

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

Gerrit-Project: flashrom
Gerrit-Branch: master
Gerrit-Change-Id: I96bb6c60e74133bbc86d5069cb076ee43f4bca83
Gerrit-Change-Number: 74522
Gerrit-PatchSet: 3
Gerrit-Owner: Thomas Heijligen <src@posteo.de>
Gerrit-Reviewer: Anastasia Klimchuk <aklm@chromium.org>
Gerrit-Reviewer: Paul Menzel <paulepanter@mailbox.org>
Gerrit-Reviewer: build bot (Jenkins) <no-reply@coreboot.org>
Gerrit-MessageType: merged