more plugin testsuite issues

Alan Modra amodra@gmail.com
Thu Oct 28 08:16:00 GMT 2010


On Tue, Oct 26, 2010 at 06:16:11PM +0100, Dave Korn wrote:
> On 26/10/2010 05:54, Alan Modra wrote:
> > Hi Dave,
> >   Is there a good reason why we need $LIBS in plugin.exp:58
> > set libs "$LIBS $HOSTING_LIBS"
> 
>   Almost certainly not.  I verified on i686-pc-cygwin and i686-pc-linux-gnu
> that nothing fails when it's removed.  Feel free to commit that change.

Committed.

	* ld-plugin/plugin.exp (libs): Remove $LIBS.

Index: ld/testsuite/ld-plugin/plugin.exp
===================================================================
RCS file: /cvs/src/src/ld/testsuite/ld-plugin/plugin.exp,v
retrieving revision 1.3
diff -u -p -r1.3 plugin.exp
--- ld/testsuite/ld-plugin/plugin.exp	20 Oct 2010 14:40:30 -0000	1.3
+++ ld/testsuite/ld-plugin/plugin.exp	28 Oct 2010 08:08:36 -0000
@@ -55,7 +55,7 @@ verbose "Full plugin path $plugin_path" 
 
 set testobjfiles "$HOSTING_CRT0 tmpdir/main.o tmpdir/func.o tmpdir/text.o"
 set testobjfiles_notext "$HOSTING_CRT0 tmpdir/main.o tmpdir/func.o"
-set libs "$LIBS $HOSTING_LIBS"
+set libs "$HOSTING_LIBS"
 
 set regclm "-plugin-opt registerclaimfile"
 set regas "-plugin-opt registerallsymbolsread"

-- 
Alan Modra
Australia Development Lab, IBM



More information about the Binutils mailing list