threads / patch / 46682

patchdiff-highlight: Add clean target to Makefile

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

## tl;dr

2 messages between Aug 29, 2017 and Aug 29, 2017. Diffs are folded; open one to read it.

replies: 1people: 2as markdown or json

Daniel Watkins· Aug 29, 2017, 11:23 UTC · lore

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(+)

Show changes to contrib/diff-highlight/Makefile +3 −0

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

Jeff King· Aug 29, 2017, 17:05 UTC · re: Daniel Watkins · lore

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

On Tue, Aug 29, 2017 at 12:23:11PM +0100, Daniel Watkins wrote:

Show 6 quoted lines

> Now that `make` produces a file, we should have a clean target to remove
> it.
> [...]
> +clean:
> +	$(RM) diff-highlight
> +

Makes sense. Thanks!

-Peff