Correct readelf thin archive test

Alan Modra amodra@gmail.com
Thu Aug 14 13:22:41 GMT 2025


If we have an existing archive, the test may fail due to it being the
wrong format.  I noticed this when messing with archive formats, a
wrong format persists and gives a failure even when you've fixed your
bug.

Also, downloading bintest.thin.a from a remote host (before creating
it!) is wrong.

	* testsuite/binutils-all/readelf.exp (readelf_thin_archive_test):
	Don't remote_download bintest.thin.a.  Delete lib before
	creating.

diff --git a/binutils/testsuite/binutils-all/readelf.exp b/binutils/testsuite/binutils-all/readelf.exp
index 571bb9986a1..22f2a355bfd 100644
--- a/binutils/testsuite/binutils-all/readelf.exp
+++ b/binutils/testsuite/binutils-all/readelf.exp
@@ -317,22 +317,21 @@ proc readelf_thin_archive_test {} {
 
     if ![is_remote host] {
 	set tempfile tmpdir/bintest.o
-        set templib  tmpdir/bintest.thin.a
         set libname  tmpdir/bintest.thin.a
     } else {
 	set tempfile [remote_download host tmpdir/bintest.o]
-        set templib  [remote_download host tmpdir/bintest.thin.a]
         set libname  bintest.thin.a
     }
 
     set testname "readelf -h bintest.thin"
+    file_on_host delete $libname
     set got [binutils_run $AR "rcT $libname ${tempfile}"]
     if ![string match "" $got] {
         fail $testname
         return
     }
 
-    readelf_test -h $templib readelf.h.thin
+    readelf_test -h $libname readelf.h.thin
 
     pass $testname
 }

-- 
Alan Modra


More information about the Binutils mailing list