Tobias Burnus wrote:
Thanks for looking at the patch. However, the patch has a link problem. The documentation is at
http://gcc.gnu.org/onlinedocs/gcc/Loop_002dSpecific-Pragmas.html

That's also the link I use in the changes.html file. However, some script changes the link to:
   http://gcc.gnu.org/onlinedocs/gcc/Loop-Specific-Pragmas.html
which won't work. Try yourself at http://gcc.gnu.org/gcc-4.9/changes.html


Actually, a similar issue was reported at http://gcc.gnu.org/ml/gcc-help/2013-10/msg00132.html

The reason for the broken links are the following lines in the /www/bin/preprocess script: http://gcc.gnu.org/cgi-bin/cvsweb.cgi/wwwdocs/bin/preprocess.diff?r1=1.38&r2=1.39&f=h

Gerald, do you still know why you added it 9 years ago? The commit comment is "Use sed to work around makeinfo 4.7 brokenness."

I think "makeinfo" is still broken, but those pages do not seem to go through the preprocess script, which means that only links to that page will change to a hyphen, breaking the links.

Do you think it would be sensible to remove those lines again - or, alternatively, to run a similar script (e.g. "perl -i -e 's/_002d/-/g' `find onlinedocs -name \*.html`) on the onlinedocs/.

I think the impact of the the former on links is smaller. (One still needs to re-run the script on those files to restore the links.)

Tobias

Reply via email to