{"thread":{"id":"46682","subject":"[PATCH] diff-highlight: Add clean target to Makefile","startedAt":"2017-08-29T11:26:16Z","lastAt":"2017-08-29T17:06:04Z","messageCount":2,"participants":["Daniel Watkins","Jeff King"],"isPatch":true,"patchVersion":1,"patchTotal":null},"messages":[{"id":"327320","messageId":"20170829112311.16432-1-daniel@daniel-watkins.co.uk","threadId":"46682","inReplyTo":null,"subject":"[PATCH] diff-highlight: Add clean target to Makefile","fromName":"Daniel Watkins","fromEmail":"daniel@daniel-watkins.co.uk","sentAt":"2017-08-29T11:23:11Z","receivedAt":"2017-08-29T11:26:16Z","isPatch":true,"sender":{"key":"daniel@daniel-watkins.co.uk","avatar":null},"body":"Now that `make` produces a file, we should have a clean target to remove\nit.\n\nSigned-off-by: Daniel Watkins <daniel@daniel-watkins.co.uk>\n---\n contrib/diff-highlight/Makefile | 3 +++\n 1 file changed, 3 insertions(+)\n\ndiff --git a/contrib/diff-highlight/Makefile b/contrib/diff-highlight/Makefile\nindex fbf5c5824..f2be7cc92 100644\n--- a/contrib/diff-highlight/Makefile\n+++ b/contrib/diff-highlight/Makefile\n@@ -17,4 +17,7 @@ shebang.perl: FORCE\n test: all\n \t$(MAKE) -C t\n \n+clean:\n+\t$(RM) diff-highlight\n+\n .PHONY: FORCE\n-- \n2.14.1\n\n"},{"id":"327325","messageId":"20170829170555.iaxvyar5widvhoso@sigill.intra.peff.net","threadId":"46682","inReplyTo":"20170829112311.16432-1-daniel@daniel-watkins.co.uk","subject":"Re: [PATCH] diff-highlight: Add clean target to Makefile","fromName":"Jeff King","fromEmail":"peff@peff.net","sentAt":"2017-08-29T17:05:56Z","receivedAt":"2017-08-29T17:06:04Z","isPatch":true,"sender":{"key":"peff@peff.net","avatar":"https://avatars.githubusercontent.com/u/45925?v=4"},"body":"On Tue, Aug 29, 2017 at 12:23:11PM +0100, Daniel Watkins wrote:\n\n> Now that `make` produces a file, we should have a clean target to remove\n> it.\n> [...]\n> +clean:\n> +\t$(RM) diff-highlight\n> +\n\nMakes sense. Thanks!\n\n-Peff\n"}]}