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

2 messages from 2017-08-29 to 2017-08-29. Participants: Daniel Watkins, Jeff King.
Thread: https://gitlist.dev/t/46682

## Daniel Watkins, 2017-08-29 11:23

Subject: [PATCH] diff-highlight: Add clean target to Makefile
Message-ID: <20170829112311.16432-1-daniel@daniel-watkins.co.uk>
URL: https://gitlist.dev/e/20170829112311.16432-1-daniel%40daniel-watkins.co.uk

```
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


```

## Jeff King, 2017-08-29 17:05

Subject: Re: [PATCH] diff-highlight: Add clean target to Makefile
Message-ID: <20170829170555.iaxvyar5widvhoso@sigill.intra.peff.net>
URL: https://gitlist.dev/e/20170829170555.iaxvyar5widvhoso%40sigill.intra.peff.net
In-Reply-To: <20170829112311.16432-1-daniel@daniel-watkins.co.uk>

```
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

```