{"thread":{"id":"27832","subject":"[PATCH] Documentation/Notes: Remove 'footnote:' warning","startedAt":"2011-07-16T22:30:20Z","lastAt":"2011-07-19T20:43:53Z","messageCount":10,"participants":["Pavan Kumar Sunkara","Drew Northup","Jonathan Nieder","Junio C Hamano"],"isPatch":true,"patchVersion":1,"patchTotal":null},"messages":[{"id":"171493","messageId":"1310855420-21183-1-git-send-email-pavan.sss1991@gmail.com","threadId":"27832","inReplyTo":null,"subject":"[PATCH] Documentation/Notes: Remove 'footnote:' warning","fromName":"Pavan Kumar Sunkara","fromEmail":"pavan.sss1991@gmail.com","sentAt":"2011-07-16T22:30:20Z","receivedAt":"2011-07-16T22:30:20Z","isPatch":true,"sender":{"key":"pavan.sss1991@gmail.com","avatar":"https://avatars.githubusercontent.com/u/174703?v=4"},"body":"Remove 'footnote:' from git-notes.txt which is responsible\nfor the warnings surfacing during the generation of git\ndocumentation.\n---\n Documentation/git-notes.txt |    9 +++++----\n 1 files changed, 5 insertions(+), 4 deletions(-)\n\ndiff --git a/Documentation/git-notes.txt b/Documentation/git-notes.txt\nindex 6a187f2..2a18894 100644\n--- a/Documentation/git-notes.txt\n+++ b/Documentation/git-notes.txt\n@@ -210,10 +210,11 @@ Commit notes are blobs containing extra information about an object\n are taken from notes refs.  A notes ref is usually a branch which\n contains \"files\" whose paths are the object names for the objects\n they describe, with some directory separators included for performance\n-reasons footnote:[Permitted pathnames have the form\n-'ab'`/`'cd'`/`'ef'`/`'...'`/`'abcdef...': a sequence of directory\n-names of two hexadecimal digits each followed by a filename with the\n-rest of the object ID.].\n+reasons.\n+\n+Permitted pathnames have the form 'ab'`/`'cd'`/`'ef'`/`'...'`/`'abcdef...':\n+a sequence of directory names of two hexadecimal digits each\n+followed by a filename with the rest of the object ID.\n \n Every notes change creates a new commit at the specified notes ref.\n You can therefore inspect the history of the notes by invoking, e.g.,\n-- \n1.7.6.485.gad3dc\n"},{"id":"171501","messageId":"1310908271.21563.17.camel@drew-northup.unet.maine.edu","threadId":"27832","inReplyTo":"1310855420-21183-1-git-send-email-pavan.sss1991@gmail.com","subject":"Re: [PATCH] Documentation/Notes: Remove 'footnote:' warning","fromName":"Drew Northup","fromEmail":"drew.northup@maine.edu","sentAt":"2011-07-17T13:11:11Z","receivedAt":"2011-07-17T13:11:11Z","isPatch":true,"sender":{"key":"drew.northup@maine.edu","avatar":"https://avatars.githubusercontent.com/u/18331571?v=4"},"body":"\nOn Sun, 2011-07-17 at 04:00 +0530, Pavan Kumar Sunkara wrote:\n> Remove 'footnote:' from git-notes.txt which is responsible\n> for the warnings surfacing during the generation of git\n> documentation.\n> ---\n\nWhat version of asciidoc are you using? Perhaps that works just fine\nwith newer versions?\n-- \n-Drew Northup\n________________________________________________\n\"As opposed to vegetable or mineral error?\"\n-John Pescatore, SANS NewsBites Vol. 12 Num. 59\n"},{"id":"171503","messageId":"CAK9CXBWfTiD6fQxptQ_W7PbKN7x+SpPy3d44=4YrO4xwc8o5Xg@mail.gmail.com","threadId":"27832","inReplyTo":"CAK9CXBUAX42gCuRLWUpuvK6SSZXv1Cn4YNoFAfjRhouzd+thcw@mail.gmail.com","subject":"Re: [PATCH] Documentation/Notes: Remove 'footnote:' warning","fromName":"Pavan Kumar Sunkara","fromEmail":"pavan.sss1991@gmail.com","sentAt":"2011-07-17T13:46:46Z","receivedAt":"2011-07-17T13:46:46Z","isPatch":true,"sender":{"key":"pavan.sss1991@gmail.com","avatar":"https://avatars.githubusercontent.com/u/174703?v=4"},"body":"I am using asciidoc 8.6.3. It is the newest available in ubuntu repository.\n\nOn Sun, Jul 17, 2011 at 6:41 PM, Drew Northup <drew.northup@maine.edu> wrote:\n>\n> On Sun, 2011-07-17 at 04:00 +0530, Pavan Kumar Sunkara wrote:\n> > Remove 'footnote:' from git-notes.txt which is responsible\n> > for the warnings surfacing during the generation of git\n> > documentation.\n> > ---\n>\n> What version of asciidoc are you using? Perhaps that works just fine\n> with newer versions?\n> --\n> -Drew Northup\n> ________________________________________________\n> \"As opposed to vegetable or mineral error?\"\n> -John Pescatore, SANS NewsBites Vol. 12 Num. 59\n>\n"},{"id":"171521","messageId":"20110717215858.GA9906@elie","threadId":"27832","inReplyTo":"1310908271.21563.17.camel@drew-northup.unet.maine.edu","subject":"Re: [PATCH] Documentation/Notes: Remove 'footnote:' warning","fromName":"Jonathan Nieder","fromEmail":"jrnieder@gmail.com","sentAt":"2011-07-17T21:58:58Z","receivedAt":"2011-07-17T21:58:58Z","isPatch":true,"sender":{"key":"jrnieder@gmail.com","avatar":"https://avatars.githubusercontent.com/u/281595?v=4"},"body":"Hi,\n\nDrew Northup wrote:\n> On Sun, 2011-07-17 at 04:00 +0530, Pavan Kumar Sunkara wrote:\n\n>> Remove 'footnote:' from git-notes.txt which is responsible\n>> for the warnings surfacing during the generation of git\n>> documentation.\n>\n> What version of asciidoc are you using? Perhaps that works just fine\n> with newer versions?\n\nGood call.  This was fixed by Docbook XSL 1.76.0:\n\n\tNorman Walsh: block.xsl; endnotes.xsl\n\n\t\tFix bug where simpara in footnote didn't work.\n\nThe symptoms are a warning and a little extra vertical space after the\nnumberal \"1.\" in the output.  I don't know if it is worth working\naround or not.\n"},{"id":"171541","messageId":"CAK9CXBVSq_j7naHzVLMj=bW8WMKq_WfkDQP4zspkK_QizC26=g@mail.gmail.com","threadId":"27832","inReplyTo":"20110717215858.GA9906@elie","subject":"Re: [PATCH] Documentation/Notes: Remove 'footnote:' warning","fromName":"Pavan Kumar Sunkara","fromEmail":"pavan.sss1991@gmail.com","sentAt":"2011-07-18T05:42:11Z","receivedAt":"2011-07-18T05:42:11Z","isPatch":true,"sender":{"key":"pavan.sss1991@gmail.com","avatar":"https://avatars.githubusercontent.com/u/174703?v=4"},"body":"On Mon, Jul 18, 2011 at 3:28 AM, Jonathan Nieder <jrnieder@gmail.com> wrote:\n> Hi,\n>\n> Drew Northup wrote:\n>> On Sun, 2011-07-17 at 04:00 +0530, Pavan Kumar Sunkara wrote:\n>\n>>> Remove 'footnote:' from git-notes.txt which is responsible\n>>> for the warnings surfacing during the generation of git\n>>> documentation.\n>>\n>> What version of asciidoc are you using? Perhaps that works just fine\n>> with newer versions?\n>\n> Good call.  This was fixed by Docbook XSL 1.76.0:\n>\n>        Norman Walsh: block.xsl; endnotes.xsl\n>\n>                Fix bug where simpara in footnote didn't work.\n>\n> The symptoms are a warning and a little extra vertical space after the\n> numberal \"1.\" in the output.  I don't know if it is worth working\n> around or not.\n>\n\nSo, using newest version will not give us a warning?\n\n\n-- \n- Pavan Kumar Sunkara\n"},{"id":"171542","messageId":"20110718055108.GA4254@elie.gateway.2wire.net","threadId":"27832","inReplyTo":"CAK9CXBVSq_j7naHzVLMj=bW8WMKq_WfkDQP4zspkK_QizC26=g@mail.gmail.com","subject":"Re: [PATCH] Documentation/Notes: Remove 'footnote:' warning","fromName":"Jonathan Nieder","fromEmail":"jrnieder@gmail.com","sentAt":"2011-07-18T05:51:08Z","receivedAt":"2011-07-18T05:51:08Z","isPatch":true,"sender":{"key":"jrnieder@gmail.com","avatar":"https://avatars.githubusercontent.com/u/281595?v=4"},"body":"Pavan Kumar Sunkara wrote:\n> On Mon, Jul 18, 2011 at 3:28 AM, Jonathan Nieder <jrnieder@gmail.com> wrote:\n\n>> Good call.  This was fixed by Docbook XSL 1.76.0:\n>>\n>>        Norman Walsh: block.xsl; endnotes.xsl\n>>\n>>                Fix bug where simpara in footnote didn't work.\n>>\n>> The symptoms are a warning and a little extra vertical space after the\n>> numberal \"1.\" in the output.  I don't know if it is worth working\n>> around or not.\n>\n> So, using newest version will not give us a warning?\n\nYes, when I run \"make -C Documentation git-notes.1\" I don't get the\nwarning or spurious blank line.  Sorry for the lack of clarity.\n(That's no reason to not to adopt a workaround if it is pleasant, of\ncourse.)\n"},{"id":"171543","messageId":"CAK9CXBV=OMRVzEjUmb6JP8nM9bQ4DqM0D7xCywxs0TUD5+scrw@mail.gmail.com","threadId":"27832","inReplyTo":"20110718055108.GA4254@elie.gateway.2wire.net","subject":"Re: [PATCH] Documentation/Notes: Remove 'footnote:' warning","fromName":"Pavan Kumar Sunkara","fromEmail":"pavan.sss1991@gmail.com","sentAt":"2011-07-18T06:07:22Z","receivedAt":"2011-07-18T06:07:22Z","isPatch":true,"sender":{"key":"pavan.sss1991@gmail.com","avatar":"https://avatars.githubusercontent.com/u/174703?v=4"},"body":">> So, using newest version will not give us a warning?\n>\n> Yes, when I run \"make -C Documentation git-notes.1\" I don't get the\n> warning or spurious blank line.  Sorry for the lack of clarity.\n> (That's no reason to not to adopt a workaround if it is pleasant, of\n> course.)\n>\n\nCurrently the documentation at\nhttp://www.kernel.org/pub/software/scm/git/docs/git-notes.html is\nshowing the footnote wrongly. This workaround will solve that.\nJonathan, ack the patch if you want to have a workaround.\n\nThanks\n-- \n- Pavan Kumar Sunkara\n"},{"id":"171544","messageId":"20110718071308.GA4745@elie.gateway.2wire.net","threadId":"27832","inReplyTo":"CAK9CXBV=OMRVzEjUmb6JP8nM9bQ4DqM0D7xCywxs0TUD5+scrw@mail.gmail.com","subject":"Re: [PATCH] Documentation/Notes: Remove 'footnote:' warning","fromName":"Jonathan Nieder","fromEmail":"jrnieder@gmail.com","sentAt":"2011-07-18T07:13:08Z","receivedAt":"2011-07-18T07:13:08Z","isPatch":true,"sender":{"key":"jrnieder@gmail.com","avatar":"https://avatars.githubusercontent.com/u/281595?v=4"},"body":"Pavan Kumar Sunkara wrote:\n\n> Currently the documentation at\n> http://www.kernel.org/pub/software/scm/git/docs/git-notes.html is\n> showing the footnote wrongly. This workaround will solve that.\n\n(For reference, the linked-to page was generated with asciidoc 8.4.5\nand renders the footnote as\n\n\t<br />[Permitted pathnames have the\n[...]\n\trest of the object ID.]<br />.\n\ninline instead of a nicely labelled link like \"[1]\" that one might\nexpect.)\n\nWeird.  That's not the Docbook XSL bug I pointed to --- the HTML\ndocumentation is generated by asciidoc directly, without any help\nfrom Docbook XSL.  And I can't reproduce it locally, either.\n*digs around in asciidoc changelog*  Ah, Asciidoc 8.5.1 introduced\nstyling of footnotes:\n\n     * Styled HTML footnotes.\n     * Added links to HTML footnote refs.\n\nUntil the kernel.org toolchain gains that feature, wouldn't a simpler\nworkaround be to put the footnote after the period, so it renders\nthere as a simple line in brackets?\n\nBarring that, it should be possible to put this information in the\ndiscussion section without disrupting the flow of the text so much,\nfor example by putting it in parentheses or making it the last\nparagraph of the discussion section.  A patch with such a change and\nmentioning in the commit message the two bugs you're working around\nwould seem sane to me.\n\nThanks,\nJonathan\n"},{"id":"171565","messageId":"CAK9CXBUjTQVsJrERKwmYT=9mBLLDu9uG9+DrFvaOaDPBUpJjzg@mail.gmail.com","threadId":"27832","inReplyTo":"20110718071308.GA4745@elie.gateway.2wire.net","subject":"Re: [PATCH] Documentation/Notes: Remove 'footnote:' warning","fromName":"Pavan Kumar Sunkara","fromEmail":"pavan.sss1991@gmail.com","sentAt":"2011-07-18T09:01:20Z","receivedAt":"2011-07-18T09:01:20Z","isPatch":true,"sender":{"key":"pavan.sss1991@gmail.com","avatar":"https://avatars.githubusercontent.com/u/174703?v=4"},"body":">\n> Barring that, it should be possible to put this information in the\n> discussion section without disrupting the flow of the text so much,\n> for example by putting it in parentheses or making it the last\n> paragraph of the discussion section.\n\nThat is what I did.\n\n> A patch with such a change and\n> mentioning in the commit message the two bugs you're working around\n> would seem sane to me.\n>\n> Thanks,\n> Jonathan\n>\n\nI agree. So, if the patch is ok, I will rewrite the commit msg and\nsend it again.\n\nThanks,\nPavan Kumar Sunkara\n\n-- \n- Pavan Kumar Sunkara\n"},{"id":"171703","messageId":"7vr55m80hi.fsf@alter.siamese.dyndns.org","threadId":"27832","inReplyTo":"CAK9CXBUjTQVsJrERKwmYT=9mBLLDu9uG9+DrFvaOaDPBUpJjzg@mail.gmail.com","subject":"Re: [PATCH] Documentation/Notes: Remove 'footnote:' warning","fromName":"Junio C Hamano","fromEmail":"gitster@pobox.com","sentAt":"2011-07-19T20:43:53Z","receivedAt":"2011-07-19T20:43:53Z","isPatch":true,"sender":{"key":"gitster@pobox.com","avatar":"https://avatars.githubusercontent.com/u/54884?v=4"},"body":"Pavan Kumar Sunkara <pavan.sss1991@gmail.com> writes:\n\n>> A patch with such a change and\n>> mentioning in the commit message the two bugs you're working around\n>> would seem sane to me.\n>>\n>> Thanks,\n>> Jonathan\n>>\n>\n> I agree. So, if the patch is ok, I will rewrite the commit msg and\n> send it again.\n>\n> Thanks,\n> Pavan Kumar Sunkara\n\nThanks.\n"}]}