public inbox for gcc@gcc.gnu.org
 help / color / mirror / Atom feed
* Broken links in INSTALL/specific.html
@ 2018-05-01  7:27 Jakub Jelinek
  2018-05-01  9:43 ` Andrew Roberts
  0 siblings, 1 reply; 2+ messages in thread
From: Jakub Jelinek @ 2018-05-01  7:27 UTC (permalink / raw)
  To: Gerald Pfeifer, Richard Biener, Joseph S. Myers; +Cc: gcc

Hi!

PR web/85578 complains about broken links in INSTALL/specific.html inside of
the rc tarballs, I've looked at past releases and at least the releases I've
checked (4.7.0, 6.1, 7.1, 7.3, 8.1rc2) all have the broken links,
e.g.
<a href="#aarch64-x-x">aarch64*-*-*</a>
and
<a name="aarch64_002dx_002dx"></a><a name="aarch64_002a_002d_002a_002d_002a"></a>
<h3 class="heading">aarch64*-*-*</h3>
Looking at online docs, they are ok.

I think this has been fixed for the online docs with:
Index: preprocess
===================================================================
RCS file: /cvs/gcc/wwwdocs/bin/preprocess,v
retrieving revision 1.38
retrieving revision 1.39
diff -u -p -r1.38 -r1.39
--- preprocess  28 Aug 2003 13:05:38 -0000      1.38
+++ preprocess  5 Sep 2004 21:50:02 -0000       1.39
@@ -144,7 +144,10 @@ process_file()
             cat $STYLE > $TMPDIR/input
             printf '<set-var MHTML::INCLUDE-PREFIX="%s">\n' `pwd` >> $TMPDIR/input
             cat $f >> $TMPDIR/input
-            ${MHC} $TMPDIR/input > $TMPDIR/output
+            # Use sed to work around makeinfo 4.7 brokenness.
+            ${MHC} $TMPDIR/input \
+                | sed -e 's/_002d/-/g' -e 's/_002a/*/g' \
+                > $TMPDIR/output
 
             # Copy the page only if it's new or there has been a change, and,
             # first of all, if there was no problem when running MetaHTML.

revision 1.39
date: 2004/09/05 21:50:02;  author: gerald;  state: Exp;  lines: +4 -1
Use sed to work around makeinfo 4.7 brokenness.

Isn't this something we should be doing in gcc/doc/install.texi2html
too (or somewhere else)?

Bugzilla is down, so can't discuss it there...

	Jakub

^ permalink raw reply	[flat|nested] 2+ messages in thread

* Re: Broken links in INSTALL/specific.html
  2018-05-01  7:27 Broken links in INSTALL/specific.html Jakub Jelinek
@ 2018-05-01  9:43 ` Andrew Roberts
  0 siblings, 0 replies; 2+ messages in thread
From: Andrew Roberts @ 2018-05-01  9:43 UTC (permalink / raw)
  To: gcc

The reason I was looking at the versions in the RC tarball was that I 
have never been clear as to what release the website 
install/prerequisite/target info actually applies to.
It would be much better if this info was on a per release basis on the 
web site, like the changelog and manuals. Thus any target
specific stuff which was removed wouldn't affect older releases 
documentation.

Speaking of manuals, it might be worth documenting the make commands for 
the documentation (make html, install-html and pdf, install-pdf) on the 
build.html page.
The documentation can only get checked before the release if people are 
aware how to build it.

Andrew


On 01/05/18 08:27, Jakub Jelinek wrote:
> Hi!
>
> PR web/85578 complains about broken links in INSTALL/specific.html inside of
> the rc tarballs, I've looked at past releases and at least the releases I've
> checked (4.7.0, 6.1, 7.1, 7.3, 8.1rc2) all have the broken links,
> e.g.
> <a href="#aarch64-x-x">aarch64*-*-*</a>
> and
> <a name="aarch64_002dx_002dx"></a><a name="aarch64_002a_002d_002a_002d_002a"></a>
> <h3 class="heading">aarch64*-*-*</h3>
> Looking at online docs, they are ok.
>
> I think this has been fixed for the online docs with:
> Index: preprocess
> ===================================================================
> RCS file: /cvs/gcc/wwwdocs/bin/preprocess,v
> retrieving revision 1.38
> retrieving revision 1.39
> diff -u -p -r1.38 -r1.39
> --- preprocess  28 Aug 2003 13:05:38 -0000      1.38
> +++ preprocess  5 Sep 2004 21:50:02 -0000       1.39
> @@ -144,7 +144,10 @@ process_file()
>               cat $STYLE > $TMPDIR/input
>               printf '<set-var MHTML::INCLUDE-PREFIX="%s">\n' `pwd` >> $TMPDIR/input
>               cat $f >> $TMPDIR/input
> -            ${MHC} $TMPDIR/input > $TMPDIR/output
> +            # Use sed to work around makeinfo 4.7 brokenness.
> +            ${MHC} $TMPDIR/input \
> +                | sed -e 's/_002d/-/g' -e 's/_002a/*/g' \
> +                > $TMPDIR/output
>   
>               # Copy the page only if it's new or there has been a change, and,
>               # first of all, if there was no problem when running MetaHTML.
>
> revision 1.39
> date: 2004/09/05 21:50:02;  author: gerald;  state: Exp;  lines: +4 -1
> Use sed to work around makeinfo 4.7 brokenness.
>
> Isn't this something we should be doing in gcc/doc/install.texi2html
> too (or somewhere else)?
>
> Bugzilla is down, so can't discuss it there...
>
> 	Jakub
>

^ permalink raw reply	[flat|nested] 2+ messages in thread

end of thread, other threads:[~2018-05-01  9:43 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2018-05-01  7:27 Broken links in INSTALL/specific.html Jakub Jelinek
2018-05-01  9:43 ` Andrew Roberts

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).