public inbox for gcc@gcc.gnu.org
 help / color / mirror / Atom feed
From: Gerald Pfeifer <gerald@pfeifer.com>
To: Patrick Marlier <patrick.marlier@gmail.com>
Cc: gcc@gcc.gnu.org
Subject: Re: [doc] Fixing reference inside Extended-Asm.html
Date: Mon, 02 Dec 2013 19:09:00 -0000	[thread overview]
Message-ID: <alpine.LNX.2.00.1312021939450.2185@tuna.site> (raw)
In-Reply-To: <CAKQMxzQY+ZNU_50D7Lda6qT94BaWCT5ZOreDTWy_MbYN0hZ52Q@mail.gmail.com>

On Wed, 20 Nov 2013, Patrick Marlier wrote:
> I have noticed that the link
> Extended-asm-with-goto.html#Extended-asm-with-goto is wrong in
> http://gcc.gnu.org/onlinedocs/gcc/Extended-Asm.html.
> Locally when I generate the html doc, the link is good, ie <a
> href="#Extended-asm-with-goto">.

I had a look, and the way makeinfo generates HTML is, hmm, creative.

For an internal reference (@xref) like this it creates a new file based 
on the reference name -- Extended-asm-with-goto.html -- and that file then 
only contains a single line which is a redirect back to the original page.  

And here is our inconsistency since it encodes blank characters as %20

  <meta http-equiv="refresh" content="0; 
  url=Extended-Asm.html#Extended%20asm%20with%20goto">

whereas in the document itself it reads

  <a name="Extended-asm-with-goto">

> Also the online page for Extended-asm-with-goto.html is wrong, '%20'
> instead of '-'.

As far as I can tell both approaches to encode blanks would be fine.

The problem is that they should be used consistently, which is not
what happens on gcc.gnu.org running makeinfo 4.13.

> Is the version of texinfo buggy to generate online documentation?

That'd be my assumption.  Which version are you using locally that
does not exhibit this behavior?

My guess is you are using a more recent version of makeinfo.  Updating 
gcc.gnu.org will be hard (since its a standard RHEL 6 system) and I
won't have the time to hack around this (applying sed magic, say).

Gerald

      parent reply	other threads:[~2013-12-02 19:09 UTC|newest]

Thread overview: 3+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2013-11-20 12:56 Patrick Marlier
2013-11-22  9:46 ` dw
2013-12-02 19:09 ` Gerald Pfeifer [this message]

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=alpine.LNX.2.00.1312021939450.2185@tuna.site \
    --to=gerald@pfeifer.com \
    --cc=gcc@gcc.gnu.org \
    --cc=patrick.marlier@gmail.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).