threads / patch / 27832

patchDocumentation/Notes: Remove 'footnote:' warning

Subject: [PATCH] Documentation/Notes: Remove 'footnote:' warning

## tl;dr

10 messages between Jul 16, 2011 and Jul 19, 2011. Diffs are folded; open one to read it.

replies: 9people: 4as markdown or json

Pavan Kumar Sunkara· Jul 16, 2011, 22:30 UTC · lore
Remove 'footnote:' from git-notes.txt which is responsible
for the warnings surfacing during the generation of git
documentation.
---
 Documentation/git-notes.txt |    9 +++++----
 1 files changed, 5 insertions(+), 4 deletions(-)
Show changes to Documentation/git-notes.txt +5 −4
diff --git a/Documentation/git-notes.txt b/Documentation/git-notes.txt
index 6a187f2..2a18894 100644
--- a/Documentation/git-notes.txt
+++ b/Documentation/git-notes.txt
@@ -210,10 +210,11 @@ Commit notes are blobs containing extra information about an object
 are taken from notes refs.  A notes ref is usually a branch which
 contains "files" whose paths are the object names for the objects
 they describe, with some directory separators included for performance
-reasons footnote:[Permitted pathnames have the form
-'ab'`/`'cd'`/`'ef'`/`'...'`/`'abcdef...': a sequence of directory
-names of two hexadecimal digits each followed by a filename with the
-rest of the object ID.].
+reasons.
+
+Permitted pathnames have the form 'ab'`/`'cd'`/`'ef'`/`'...'`/`'abcdef...':
+a sequence of directory names of two hexadecimal digits each
+followed by a filename with the rest of the object ID.
 
 Every notes change creates a new commit at the specified notes ref.
 You can therefore inspect the history of the notes by invoking, e.g.,
-- 
1.7.6.485.gad3dc
Drew Northup· Jul 17, 2011, 13:11 UTC · re: Pavan Kumar Sunkara · lore

Re: [PATCH] Documentation/Notes: Remove 'footnote:' warning

On Sun, 2011-07-17 at 04:00 +0530, Pavan Kumar Sunkara wrote:
> Remove 'footnote:' from git-notes.txt which is responsible
> for the warnings surfacing during the generation of git
> documentation.
> ---

What version of asciidoc are you using? Perhaps that works just fine with newer versions?

-- 
-Drew Northup
________________________________________________
"As opposed to vegetable or mineral error?"
-John Pescatore, SANS NewsBites Vol. 12 Num. 59
Jonathan Nieder· Jul 17, 2011, 21:58 UTC · re: Drew Northup · lore

Re: [PATCH] Documentation/Notes: Remove 'footnote:' warning

Hi,
Drew Northup wrote:
> On Sun, 2011-07-17 at 04:00 +0530, Pavan Kumar Sunkara wrote:
Show 6 quoted lines
>> Remove 'footnote:' from git-notes.txt which is responsible
>> for the warnings surfacing during the generation of git
>> documentation.
>
> What version of asciidoc are you using? Perhaps that works just fine
> with newer versions?
Good call.  This was fixed by Docbook XSL 1.76.0:
	Norman Walsh: block.xsl; endnotes.xsl
		Fix bug where simpara in footnote didn't work.

The symptoms are a warning and a little extra vertical space after the numberal "1." in the output. I don't know if it is worth working around or not.

Pavan Kumar Sunkara· Jul 18, 2011, 05:42 UTC · re: Jonathan Nieder · lore

Re: [PATCH] Documentation/Notes: Remove 'footnote:' warning

On Mon, Jul 18, 2011 at 3:28 AM, Jonathan Nieder <jrnieder@gmail.com> wrote:
Show 22 quoted lines
> Hi,
>
> Drew Northup wrote:
>> On Sun, 2011-07-17 at 04:00 +0530, Pavan Kumar Sunkara wrote:
>
>>> Remove 'footnote:' from git-notes.txt which is responsible
>>> for the warnings surfacing during the generation of git
>>> documentation.
>>
>> What version of asciidoc are you using? Perhaps that works just fine
>> with newer versions?
>
> Good call.  This was fixed by Docbook XSL 1.76.0:
>
>        Norman Walsh: block.xsl; endnotes.xsl
>
>                Fix bug where simpara in footnote didn't work.
>
> The symptoms are a warning and a little extra vertical space after the
> numberal "1." in the output.  I don't know if it is worth working
> around or not.
>
So, using newest version will not give us a warning?
-- 
- Pavan Kumar Sunkara
Jonathan Nieder· Jul 18, 2011, 05:51 UTC · re: Pavan Kumar Sunkara · lore

Re: [PATCH] Documentation/Notes: Remove 'footnote:' warning

Pavan Kumar Sunkara wrote:
> On Mon, Jul 18, 2011 at 3:28 AM, Jonathan Nieder <jrnieder@gmail.com> wrote:
Show 11 quoted lines
>> Good call.  This was fixed by Docbook XSL 1.76.0:
>>
>>        Norman Walsh: block.xsl; endnotes.xsl
>>
>>                Fix bug where simpara in footnote didn't work.
>>
>> The symptoms are a warning and a little extra vertical space after the
>> numberal "1." in the output.  I don't know if it is worth working
>> around or not.
>
> So, using newest version will not give us a warning?

Yes, when I run "make -C Documentation git-notes.1" I don't get the warning or spurious blank line. Sorry for the lack of clarity. (That's no reason to not to adopt a workaround if it is pleasant, of course.)

Pavan Kumar Sunkara· Jul 18, 2011, 06:07 UTC · re: Jonathan Nieder · lore

Re: [PATCH] Documentation/Notes: Remove 'footnote:' warning

Show 7 quoted lines
>> So, using newest version will not give us a warning?
>
> Yes, when I run "make -C Documentation git-notes.1" I don't get the
> warning or spurious blank line.  Sorry for the lack of clarity.
> (That's no reason to not to adopt a workaround if it is pleasant, of
> course.)
>

Currently the documentation at http://www.kernel.org/pub/software/scm/git/docs/git-notes.html is showing the footnote wrongly. This workaround will solve that. Jonathan, ack the patch if you want to have a workaround.

Thanks
-- 
- Pavan Kumar Sunkara
Jonathan Nieder· Jul 18, 2011, 07:13 UTC · re: Pavan Kumar Sunkara · lore

Re: [PATCH] Documentation/Notes: Remove 'footnote:' warning

Pavan Kumar Sunkara wrote:
> Currently the documentation at
> http://www.kernel.org/pub/software/scm/git/docs/git-notes.html is
> showing the footnote wrongly. This workaround will solve that.

(For reference, the linked-to page was generated with asciidoc 8.4.5 and renders the footnote as

	<br />[Permitted pathnames have the
[...]
	rest of the object ID.]<br />.

inline instead of a nicely labelled link like "[1]" that one might expect.)

Weird. That's not the Docbook XSL bug I pointed to --- the HTML documentation is generated by asciidoc directly, without any help from Docbook XSL. And I can't reproduce it locally, either. *digs around in asciidoc changelog* Ah, Asciidoc 8.5.1 introduced styling of footnotes:

     * Styled HTML footnotes.
     * Added links to HTML footnote refs.

Until the kernel.org toolchain gains that feature, wouldn't a simpler workaround be to put the footnote after the period, so it renders there as a simple line in brackets?

Barring that, it should be possible to put this information in the discussion section without disrupting the flow of the text so much, for example by putting it in parentheses or making it the last paragraph of the discussion section. A patch with such a change and mentioning in the commit message the two bugs you're working around would seem sane to me.

Thanks, Jonathan

Pavan Kumar Sunkara· Jul 18, 2011, 09:01 UTC · re: Jonathan Nieder · lore

Re: [PATCH] Documentation/Notes: Remove 'footnote:' warning

Show 5 quoted lines
>
> Barring that, it should be possible to put this information in the
> discussion section without disrupting the flow of the text so much,
> for example by putting it in parentheses or making it the last
> paragraph of the discussion section.
That is what I did.
Show 7 quoted lines
> A patch with such a change and
> mentioning in the commit message the two bugs you're working around
> would seem sane to me.
>
> Thanks,
> Jonathan
>

I agree. So, if the patch is ok, I will rewrite the commit msg and send it again.

Thanks, Pavan Kumar Sunkara

-- 
- Pavan Kumar Sunkara
Junio C Hamano· Jul 19, 2011, 20:43 UTC · re: Pavan Kumar Sunkara · lore

Re: [PATCH] Documentation/Notes: Remove 'footnote:' warning

Pavan Kumar Sunkara <pavan.sss1991@gmail.com> writes:
Show 13 quoted lines
>> A patch with such a change and
>> mentioning in the commit message the two bugs you're working around
>> would seem sane to me.
>>
>> Thanks,
>> Jonathan
>>
>
> I agree. So, if the patch is ok, I will rewrite the commit msg and
> send it again.
>
> Thanks,
> Pavan Kumar Sunkara
Thanks.
Pavan Kumar Sunkara· Jul 17, 2011, 13:46 UTC · lore

Re: [PATCH] Documentation/Notes: Remove 'footnote:' warning

I am using asciidoc 8.6.3. It is the newest available in ubuntu repository.
On Sun, Jul 17, 2011 at 6:41 PM, Drew Northup <drew.northup@maine.edu> wrote:
Show 15 quoted lines
>
> On Sun, 2011-07-17 at 04:00 +0530, Pavan Kumar Sunkara wrote:
> > Remove 'footnote:' from git-notes.txt which is responsible
> > for the warnings surfacing during the generation of git
> > documentation.
> > ---
>
> What version of asciidoc are you using? Perhaps that works just fine
> with newer versions?
> --
> -Drew Northup
> ________________________________________________
> "As opposed to vegetable or mineral error?"
> -John Pescatore, SANS NewsBites Vol. 12 Num. 59
>

← back to recent threads