[PATCH] diff-highlight: Add clean target to Makefile

Subsystems: the rest

STALE3260d

2 messages, 2 authors, 2017-08-29 · open the first message on its own page

[PATCH] diff-highlight: Add clean target to Makefile

From: Daniel Watkins <hidden>
Date: 2017-08-29 11:26:16

Now that `make` produces a file, we should have a clean target to remove
it.

Signed-off-by: Daniel Watkins <redacted>
---
 contrib/diff-highlight/Makefile | 3 +++
 1 file changed, 3 insertions(+)
diff --git a/contrib/diff-highlight/Makefile b/contrib/diff-highlight/Makefile
index fbf5c5824..f2be7cc92 100644
--- a/contrib/diff-highlight/Makefile
+++ b/contrib/diff-highlight/Makefile
@@ -17,4 +17,7 @@ shebang.perl: FORCE
 test: all
 	$(MAKE) -C t
 
+clean:
+	$(RM) diff-highlight
+
 .PHONY: FORCE
-- 
2.14.1

Re: [PATCH] diff-highlight: Add clean target to Makefile

From: Jeff King <hidden>
Date: 2017-08-29 17:06:04

On Tue, Aug 29, 2017 at 12:23:11PM +0100, Daniel Watkins wrote:
Now that `make` produces a file, we should have a clean target to remove
it.
[...]
+clean:
+	$(RM) diff-highlight
+
Makes sense. Thanks!

-Peff
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help