[coreboot-gerrit] Change in coreboot[master]: payloads/depthcharge: Fix clean target

Nico Huber (Code Review) gerrit at coreboot.org
Tue Jul 25 16:53:56 CEST 2017


Nico Huber has uploaded this change for review. ( https://review.coreboot.org/20769


Change subject: payloads/depthcharge: Fix clean target
......................................................................

payloads/depthcharge: Fix clean target

This line has a weird history. It got a `|| exit 0` removed lately which
obviously was there to fix the presence of the superfluous `test` at the
beginning. Now, remove the `test` too to make the clean target always
succeed again ;)

Change-Id: I9e069cf5d9ac8416cf350161439aa60798ef7b6b
Signed-off-by: Nico Huber <nico.huber at secunet.com>
---
M payloads/external/depthcharge/Makefile
1 file changed, 1 insertion(+), 1 deletion(-)



  git pull ssh://review.coreboot.org:29418/coreboot refs/changes/69/20769/1

diff --git a/payloads/external/depthcharge/Makefile b/payloads/external/depthcharge/Makefile
index 1c8d345..8115eb1 100644
--- a/payloads/external/depthcharge/Makefile
+++ b/payloads/external/depthcharge/Makefile
@@ -89,7 +89,7 @@
 		PATH="$(abspath ../../../build/util/cbfstool):$$PATH"
 
 clean:
-	test -d $(output_dir) && rm -rf $(output_dir)
+	rm -rf $(output_dir)
 
 distclean:
 	rm -rf $(project_dir)

-- 
To view, visit https://review.coreboot.org/20769
To unsubscribe, visit https://review.coreboot.org/settings

Gerrit-Project: coreboot
Gerrit-Branch: master
Gerrit-MessageType: newchange
Gerrit-Change-Id: I9e069cf5d9ac8416cf350161439aa60798ef7b6b
Gerrit-Change-Number: 20769
Gerrit-PatchSet: 1
Gerrit-Owner: Nico Huber <nico.h at gmx.de>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.coreboot.org/pipermail/coreboot-gerrit/attachments/20170725/8139ab26/attachment.html>


More information about the coreboot-gerrit mailing list