{"thread":{"id":"32333","subject":"[PATCH] For git-subtree, when installing docs (make install-doc), create man1 folder first.","startedAt":"2012-12-13T20:09:36Z","lastAt":"2013-01-01T03:06:08Z","messageCount":4,"participants":["Jesper L. Nielsen","Junio C Hamano","greened@obbligato.org"],"isPatch":true,"patchVersion":1,"patchTotal":null},"messages":[{"id":"204851","messageId":"1355429376-4192-1-git-send-email-lyager@gmail.com","threadId":"32333","inReplyTo":null,"subject":"[PATCH] For git-subtree, when installing docs (make install-doc), create man1 folder first.","fromName":"Jesper L. Nielsen","fromEmail":"lyager@gmail.com","sentAt":"2012-12-13T20:09:36Z","receivedAt":"2012-12-13T20:09:36Z","isPatch":true,"sender":{"key":"lyager@gmail.com","avatar":null},"body":"From: \"Jesper L. Nielsen\" \n\nHi..\n\nI installed Git subtree and discovered that the if the man1dir doesn't exist the man-page for Git Subtree is just called man1.\n\nSo, small patch to create the folder first in the Makefile. Hope everything is right with the patch and submitting of the patch.\n\nBest Regards\nJesper\n\nSigned-off-by: Jesper L. Nielsen \n---\n contrib/subtree/Makefile | 1 +\n 1 file changed, 1 insertion(+)\n\ndiff --git a/contrib/subtree/Makefile b/contrib/subtree/Makefile\nindex 05cdd5c..a341cf4 100644\n--- a/contrib/subtree/Makefile\n+++ b/contrib/subtree/Makefile\n@@ -35,6 +35,7 @@ install: $(GIT_SUBTREE)\n install-doc: install-man\n \n install-man: $(GIT_SUBTREE_DOC)\n+\tmkdir -p $(man1dir)\n \t$(INSTALL) -m 644 $^ $(man1dir)\n \n $(GIT_SUBTREE_DOC): $(GIT_SUBTREE_XML)\n-- \n1.8.0.2\n"},{"id":"204927","messageId":"7vlicztcjd.fsf@alter.siamese.dyndns.org","threadId":"32333","inReplyTo":"1355429376-4192-1-git-send-email-lyager@gmail.com","subject":"Re: [PATCH] For git-subtree, when installing docs (make install-doc), create man1 folder first.","fromName":"Junio C Hamano","fromEmail":"gitster@pobox.com","sentAt":"2012-12-15T18:07:02Z","receivedAt":"2012-12-15T18:07:02Z","isPatch":true,"sender":{"key":"gitster@pobox.com","avatar":"https://avatars.githubusercontent.com/u/54884?v=4"},"body":"\"Jesper L. Nielsen\" writes:\n\n> From: \"Jesper L. Nielsen\" \n>\n> Hi..\n>\n> I installed Git subtree and discovered that the if the man1dir doesn't exist the man-page for Git Subtree is just called man1.\n>\n> So, small patch to create the folder first in the Makefile. Hope everything is right with the patch and submitting of the patch.\n>\n> Best Regards\n> Jesper\n>\n> Signed-off-by: Jesper L. Nielsen \n> ---\n> contrib/subtree/Makefile | 1 +\n> 1 file changed, 1 insertion(+)\n\nThis is a good first step in the right direction, I think.\n\nShouldn't the install targets take DESTDIR into account like the\nmain Makefile does, though?\n\nAs to the patch submission, everything below \"---\" line looks sane,\nbut please check how others write commit log messages by browsing a\nhandful of entries in \"git log --no-merges\" output and notice the\ndifference (the details are found in Documentation/SubmittingPatches).\n\nThanks.\n\n> diff --git a/contrib/subtree/Makefile b/contrib/subtree/Makefile\n> index 05cdd5c..a341cf4 100644\n> --- a/contrib/subtree/Makefile\n> +++ b/contrib/subtree/Makefile\n> @@ -35,6 +35,7 @@ install: $(GIT_SUBTREE)\n> install-doc: install-man\n> \n> install-man: $(GIT_SUBTREE_DOC)\n> +\tmkdir -p $(man1dir)\n> \t$(INSTALL) -m 644 $^ $(man1dir)\n> \n> $(GIT_SUBTREE_DOC): $(GIT_SUBTREE_XML)\n"},{"id":"205739","messageId":"87623hy5bz.fsf@waller.obbligato.org","threadId":"32333","inReplyTo":"7v8v91y97f.fsf@alter.siamese.dyndns.org","subject":"Re: [PATCH] For git-subtree, when installing docs (make install-doc), create man1 folder first.","fromName":"","fromEmail":"greened@obbligato.org","sentAt":"2013-01-01T03:01:20Z","receivedAt":"2013-01-01T03:01:20Z","isPatch":true,"sender":{"key":"greened@obbligato.org","avatar":"https://avatars.githubusercontent.com/u/5291869?v=4"},"body":"Junio C Hamano writes:\n\n> \"Jesper L. Nielsen\" writes:\n>\n>> From: \"Jesper L. Nielsen\" \n>>\n>> Hi..\n>>\n>> I installed Git subtree and discovered that the if the man1dir doesn't exist the man-page for Git Subtree is just called man1.\n>>\n>> So, small patch to create the folder first in the Makefile. Hope everything is right with the patch and submitting of the patch.\n>>\n>> Best Regards\n>> Jesper\n>>\n>> Signed-off-by: Jesper L. Nielsen \n>> ---\n>\n> Thanks. David, Ack?\n\nYep. I can add it to my queue which I'll send to you tomorrow.\nOtherwise, feel free to apply it to your copy.\n\n -David\n"},{"id":"205741","messageId":"87zk0twqjj.fsf@waller.obbligato.org","threadId":"32333","inReplyTo":"1355429376-4192-1-git-send-email-lyager@gmail.com","subject":"Re: [PATCH] For git-subtree, when installing docs (make install-doc), create man1 folder first.","fromName":"","fromEmail":"greened@obbligato.org","sentAt":"2013-01-01T03:06:08Z","receivedAt":"2013-01-01T03:06:08Z","isPatch":true,"sender":{"key":"greened@obbligato.org","avatar":"https://avatars.githubusercontent.com/u/5291869?v=4"},"body":"\"Jesper L. Nielsen\" writes:\n\n> So, small patch to create the folder first in the Makefile. Hope\n> everything is right with the patch and submitting of the patch.\n\nI've applied this to my local copy and will send it to the list for\nintegration.\n\nThanks for fixing this!\n\n -David\n"}]}