{"thread":{"id":"9346","subject":"[PATCH] When generating manpages, delete outdated targets first.","startedAt":"2007-08-01T23:48:44Z","lastAt":"2007-08-02T00:53:31Z","messageCount":2,"participants":["David Kastrup","Junio C Hamano"],"isPatch":true,"patchVersion":1,"patchTotal":null},"messages":[{"id":"49376","messageId":"S1753674AbXHBABb/20070802000131Z+363@vger.kernel.org","threadId":"9346","inReplyTo":null,"subject":"[PATCH] When generating manpages, delete outdated targets first.","fromName":"David Kastrup","fromEmail":"dak@gnu.org","sentAt":"2007-08-01T23:48:44Z","receivedAt":"2007-08-01T23:48:44Z","isPatch":true,"sender":{"key":"dak@gnu.org","avatar":"https://avatars.githubusercontent.com/u/52141349?v=4"},"body":"This makes \"make doc\" work even if you made \"sudo make doc\" previously\nby mistake.  Apparently an oversight: the other targets did this already.\n\nSigned-off-by: David Kastrup <dak@gnu.org>\n---\n Documentation/Makefile |    1 +\n 1 files changed, 1 insertions(+), 0 deletions(-)\n\ndiff --git a/Documentation/Makefile b/Documentation/Makefile\nindex 3bc5357..a186843 100644\n--- a/Documentation/Makefile\n+++ b/Documentation/Makefile\n@@ -120,6 +120,7 @@ clean:\n \tmv $@+ $@\n \n %.1 %.5 %.7 : %.xml\n+\t$(RM) $@+ $@\n \txmlto -m callouts.xsl man $<\n \n %.xml : %.txt\n-- \n1.5.3.rc2.167.gbef44-dirty\n"},{"id":"49379","messageId":"7vlkcu3gfo.fsf@assigned-by-dhcp.cox.net","threadId":"9346","inReplyTo":"S1753674AbXHBABb/20070802000131Z+363@vger.kernel.org","subject":"Re: [PATCH] When generating manpages, delete outdated targets first.","fromName":"Junio C Hamano","fromEmail":"gitster@pobox.com","sentAt":"2007-08-02T00:53:31Z","receivedAt":"2007-08-02T00:53:31Z","isPatch":true,"sender":{"key":"gitster@pobox.com","avatar":"https://avatars.githubusercontent.com/u/54884?v=4"},"body":"David Kastrup <dak@gnu.org> writes:\n\n> This makes \"make doc\" work even if you made \"sudo make doc\" previously\n> by mistake.  Apparently an oversight: the other targets did this already.\n>\n> Signed-off-by: David Kastrup <dak@gnu.org>\n> ---\n>  Documentation/Makefile |    1 +\n>  1 files changed, 1 insertions(+), 0 deletions(-)\n>\n> diff --git a/Documentation/Makefile b/Documentation/Makefile\n> index 3bc5357..a186843 100644\n> --- a/Documentation/Makefile\n> +++ b/Documentation/Makefile\n> @@ -120,6 +120,7 @@ clean:\n>  \tmv $@+ $@\n>  \n>  %.1 %.5 %.7 : %.xml\n> +\t$(RM) $@+ $@\n>  \txmlto -m callouts.xsl man $<\n>  \n>  %.xml : %.txt\n\nI think \"rm\" makes sense; the call to xmlto in this Makefile has\nbeen this way from day one, it seems ;-).\n\nThe build rule for this target does not use \"generate to\ntemporary $@+ and after that is successful mv to the final $@\"\npattern, so we do not have to remove $@+ in this target, though.\n\nThanks.\n"}]}