{"thread":{"id":"43585","subject":"Re: [PATCH] no need to install manpages as executable","startedAt":"2006-12-11T07:39:32Z","lastAt":"2006-12-12T01:15:50Z","messageCount":2,"participants":["Junio C Hamano","Chris Wright"],"isPatch":true,"patchVersion":1,"patchTotal":null},"messages":[{"id":"296134","messageId":"20061211073932.GA12555@sequoia.sous-sol.org","threadId":"43585","inReplyTo":null,"subject":"[PATCH] no need to install manpages as executable","fromName":"Chris Wright","fromEmail":"chrisw@sous-sol.org","sentAt":"2006-12-11T07:39:32Z","receivedAt":"2006-12-11T07:39:32Z","isPatch":true,"sender":{"key":"chrisw@sous-sol.org","avatar":null},"body":"No need to install manpages as executable.  Noticed by Ville Skyttä.\n\nSigned-off-by: Chris Wright <chrisw@sous-sol.org>\n---\nApplies cleanly to maint and master.\n\n Documentation/Makefile |    4 ++--\n 1 file changed, 2 insertions(+), 2 deletions(-)\n\ndiff --git a/Documentation/Makefile b/Documentation/Makefile\nindex c00f5f6..d68bc4a 100644\n--- a/Documentation/Makefile\n+++ b/Documentation/Makefile\n@@ -56,8 +56,8 @@ man7: $(DOC_MAN7)\n \n install: man\n \t$(INSTALL) -d -m755 $(DESTDIR)$(man1dir) $(DESTDIR)$(man7dir)\n-\t$(INSTALL) $(DOC_MAN1) $(DESTDIR)$(man1dir)\n-\t$(INSTALL) $(DOC_MAN7) $(DESTDIR)$(man7dir)\n+\t$(INSTALL) -m644 $(DOC_MAN1) $(DESTDIR)$(man1dir)\n+\t$(INSTALL) -m644 $(DOC_MAN7) $(DESTDIR)$(man7dir)\n \n \n"},{"id":"295722","messageId":"7virghzzg9.fsf@assigned-by-dhcp.cox.net","threadId":"43585","inReplyTo":"20061211073932.GA12555@sequoia.sous-sol.org","subject":"Re: [PATCH] no need to install manpages as executable","fromName":"Junio C Hamano","fromEmail":"junkio@cox.net","sentAt":"2006-12-12T01:15:50Z","receivedAt":"2006-12-12T01:15:50Z","isPatch":true,"sender":{"key":"gitster@pobox.com","avatar":"https://avatars.githubusercontent.com/u/54884?v=4"},"body":"Chris Wright <chrisw@sous-sol.org> writes:\n\n> No need to install manpages as executable.  Noticed by Ville Skyttä.\n\nThanks.  I agree there is no need but I wonder why we even need\nto specify that explicitly.  Well, I guess I did not know that\ninstall(1) was that stupid.\n"}]}