Moved -qnolinkcmds option so arguments passed on the command line are

interpreted at the right point per Eric Norum's discovery:

    The problem with the my previous `fix' for adding linker commands
    was with the ordering of the options to the linker.

    For example, to make a larger heap size, the application Makefile would
    CFLAGS_LD = -Wl,--defsym -Wl,HeapSize=0x40000

    The command passed to the linker would be :
    m68k-rtems-ld .... -T xxx/linkcmds .... --defsym HeapSize=0x40000 .....

    This doesn't work because the script in linkcmds inserts a default
    value for HeapSize if HeapSize is not defined by the time the linker
    looks at the linkcmds script.

    The solution seems to be to move the -T linkcmds%s in the bsp_specs
    file out of the link specfication and into the lib specification -- a
    little unorthodox, perhaps, but it seems to work!

1 files changed, 2 insertions, 2 deletions

diff --git a/c/src/lib/libbsp/i386/pc386/bsp_specs b/c/src/lib/libbsp/i386/pc386/bsp_specs
index 4f95da2715..5e84733426 100644
--- a/c/src/lib/libbsp/i386/pc386/bsp_specs
+++ b/c/src/lib/libbsp/i386/pc386/bsp_specs
@@ -8,11 +8,11 @@
 %(old_cpp) %{qrtems: -D__embedded__} -Asystem(embedded)
 
 *lib:
-%{!qrtems: %(old_lib)} %{qrtems: --start-group -lrtemsall -lc -lgcc --end-group}
+%{!qrtems: %(old_lib)} %{qrtems: --start-group -lrtemsall -lc -lgcc --end-group %{!qnolinkcmds: -T linkcmds%s}}
 
 *startfile:
 %{!qrtems: %(old_startfile)} %{qrtems: start.o%s}
 
 *link:
-%{!qrtems: %(old_link)} %{qrtems: -dc -dp -N -e start %{!qnolinkcmds: -T linkcmds%s}}
+%{!qrtems: %(old_link)} %{qrtems: -dc -dp -N -e start}