* [PATCH] diff-highlight: Add clean target to Makefile
@ 2017-08-29 11:23 Daniel Watkins
2017-08-29 17:05 ` Jeff King
0 siblings, 1 reply; 2+ messages in thread
From: Daniel Watkins @ 2017-08-29 11:23 UTC (permalink / raw)
To: git; +Cc: peff, Daniel Watkins
Now that `make` produces a file, we should have a clean target to remove
it.
Signed-off-by: Daniel Watkins <daniel@daniel-watkins.co.uk>
---
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
^ permalink raw reply related [flat|nested] 2+ messages in thread
* Re: [PATCH] diff-highlight: Add clean target to Makefile
2017-08-29 11:23 [PATCH] diff-highlight: Add clean target to Makefile Daniel Watkins
@ 2017-08-29 17:05 ` Jeff King
0 siblings, 0 replies; 2+ messages in thread
From: Jeff King @ 2017-08-29 17:05 UTC (permalink / raw)
To: Daniel Watkins; +Cc: git
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
^ permalink raw reply [flat|nested] 2+ messages in thread
end of thread, other threads:[~2017-08-29 17:06 UTC | newest]
Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2017-08-29 11:23 [PATCH] diff-highlight: Add clean target to Makefile Daniel Watkins
2017-08-29 17:05 ` Jeff King
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.