diff mbox

[committed] Remove external links that texinfo would shred.

Message ID eba0e545-cd2f-b200-947f-c68ede74a5af@gjlay.de
State New
Headers show

Commit Message

Georg-Johann Lay July 11, 2017, 10:29 a.m. UTC 
texinfo is shredding external links.

Applied the following patch to prevent uses from 404 not found.

Johann

gcc/
	* doc/extend.texi (AVR Function Attributes): Remove weblink to
	Binutils doc as TEXI will mess them up.
	* doc/invoke.texi (AVR Options): Same here.


  function attribute.  This feature is activated per default

Comments

Gerald Pfeifer Dec. 17, 2017, 8:23 p.m. UTC | #1 
On Tue, 11 Jul 2017, Georg-Johann Lay wrote:
> texinfo is shredding external links.
> 
> Applied the following patch to prevent uses from 404 not found.
> 
> Johann
> 
> gcc/
> 	* doc/extend.texi (AVR Function Attributes): Remove weblink to
> 	Binutils doc as TEXI will mess them up.
> 	* doc/invoke.texi (AVR Options): Same here.

Strictly speaking it's not makeinfo, but post-processing I put in place 
many years ago to compensate what _was_ a silly change in makeinfo which
started generating "_002d" instead of "-" (and similar substitutions).

In other words, strict correctness was valued over usability and 
backwards-compatibility, let alone intuition or esthetics. </rant>

Unfortunately the workaround has been causing some collateral damage,
too, one of which your patch avoided.

Sorry, and thanks!

Gerald
diff mbox

Patch

Index: doc/extend.texi
===================================================================
--- doc/extend.texi     (revision 250123)
+++ doc/extend.texi     (working copy)
@@ -3820,8 +3820,6 @@  depended upon to work reliably and are n
  Do not use @code{__gcc_isr} pseudo instructions in a function with
  the @code{interrupt} or @code{signal} attribute aka. interrupt
  service routine (ISR).
-For details on @code{__gcc_isr}, see the GNU Binutils
-@w{@uref{https://sourceware.org/binutils/docs/as/AVR_002dDependent.html,AVR 
assembler manual}}.
  Use this attribute if the preamble of the ISR prologue should always read
  @example
  push  __zero_reg__
Index: doc/invoke.texi
===================================================================
--- doc/invoke.texi     (revision 250123)
+++ doc/invoke.texi     (working copy)
@@ -15981,9 +15981,7 @@  subroutines.  Code size is smaller.
  @item -mgas-isr-prologues
  @opindex mgas-isr-prologues
  Interrupt service routines (ISRs) may use the @code{__gcc_isr} pseudo
-instruction supported by GNU Binutils, see the
-@w{@uref{https://sourceware.org/binutils/docs/as/AVR_002dDependent.html,AVR 
assembler manual}}
-for details.
+instruction supported by GNU Binutils.
  If this option is on, the feature can still be disabled for individual
  ISRs by means of the @ref{AVR Function Attributes,,@code{no_gccisr}}