[binutils-gdb/binutils-2_45-branch] PR 33197 [AVR] Incorrect syntax in generated ldscript

[Georg-Johann Lay]

https://sourceware.org/git/gitweb.cgi?p=binutils-gdb.git;h=44cd1f92e5138d3a4f0d3387203017ccd2fd6678

commit 44cd1f92e5138d3a4f0d3387203017ccd2fd6678
Author: Alan Modra <amodra@gmail.com>
Date:   Wed Jul 23 21:52:31 2025 +0930

    PR 33197 [AVR] Incorrect syntax in generated ldscript
    
    Rearrange scripttempl/avr.sc to avoid oddities of shells expanding
    ${RELOCATING+stuff} in here documents where "stuff" contains quoted
    strings.  Also I think it is better to avoid multi-line "stuff" as it
    can be tricky to spot the ending brace.
    
    (cherry picked from commit ae114fb523efe908f9e807359e2f494ee64d2801)

Diff:
---
 ld/scripttempl/avr.sc | 22 ++++++++++++----------
 1 file changed, 12 insertions(+), 10 deletions(-)

diff --git a/ld/scripttempl/avr.sc b/ld/scripttempl/avr.sc
index 4d9af7fcf2c..a902385a013 100644
--- a/ld/scripttempl/avr.sc
+++ b/ld/scripttempl/avr.sc
@@ -186,7 +186,9 @@ SECTIONS
   /* Internal text space or external memory.  */
   .text ${RELOCATING-0} :
   {
-    ${RELOCATING+*(.vectors)
+EOF
+test -z "${RELOCATING}" || cat <<EOF
+    *(.vectors)
     KEEP(*(.vectors))
 
     /* For data that needs to reside in the lower 64k of progmem.
@@ -197,7 +199,7 @@ SECTIONS
     *(.progmem.*)
     __progmem_end = . ;
     ASSERT (__progmem_start == __progmem_end || __progmem_end <= 0x10000,
-            \".progmem section exceeds 0x10000\");
+	    ".progmem section exceeds 0x10000");
 
     . = ALIGN(2);
     __trampolines_start = . ;
@@ -210,8 +212,9 @@ SECTIONS
 
     /* For code that needs to reside in the lower 128k progmem.  */
     *(.lowtext)
-    *(.lowtext*)}
-
+    *(.lowtext*)
+EOF
+cat <<EOF
     ${CONSTRUCTING+ __ctors_start = . ; }
     ${CONSTRUCTING+ *(.ctors) }
     ${CONSTRUCTING+ __ctors_end = . ; }
@@ -365,21 +368,20 @@ EOF
 fi
 
 if test -n "${HAVE_FLMAP}"; then
-    cat <<EOF
-
-${RELOCATING+
+test -z "${RELOCATING}" || cat <<EOF
 __flmap_init_label = DEFINED(__flmap_init_start) ? __flmap_init_start : 0 ;
 /* User can specify position of .rodata in flash (LMA) by supplying
    __RODATA_FLASH_START__ or __flmap, where the former takes precedence. */
 __RODATA_FLASH_START__ = DEFINED(__RODATA_FLASH_START__)
    ? __RODATA_FLASH_START__
    : DEFINED(__flmap) ? __flmap * 32K : ${RODATA_FLASH_START};
-ASSERT (__RODATA_FLASH_START__ % 32K == 0, \"__RODATA_FLASH_START__ must be a multiple of 32 KiB\")
+ASSERT (__RODATA_FLASH_START__ % 32K == 0, "__RODATA_FLASH_START__ must be a multiple of 32 KiB")
 __flmap = ${FLMAP_MASK} & (__RODATA_FLASH_START__ >> 15);
 __RODATA_FLASH_START__ = __flmap << 15;
 __rodata_load_start = MAX (__data_load_end, __RODATA_FLASH_START__);
-__rodata_start = __RODATA_ORIGIN__ + __rodata_load_start - __RODATA_FLASH_START__;}
-
+__rodata_start = __RODATA_ORIGIN__ + __rodata_load_start - __RODATA_FLASH_START__;
+EOF
+cat << EOF
   .rodata ${RELOCATING+ __rodata_start} ${RELOCATING-0} : ${RELOCATING+ AT (__rodata_load_start)}
   {
     *(.rodata)