public inbox for binutils@sourceware.org
 help / color / mirror / Atom feed
* [PATCH/RFA]: Tidy up names of ld-script REGION_ALIAS tests
@ 2010-11-29 17:11 Matthew Gretton-Dann
  2010-11-30 10:16 ` Alan Modra
  0 siblings, 1 reply; 2+ messages in thread
From: Matthew Gretton-Dann @ 2010-11-29 17:11 UTC (permalink / raw)
  To: binutils

[-- Attachment #1: Type: text/plain, Size: 609 bytes --]

All,

The attached patch changes the names of the ld-script REGION_ALIAS tests
so that the full path of the test being run does not appear in the test
name.

This makes it easier to diff two testsuite runs based in different
directories.

The patch has been tested on arm-none-eabi.

Please can someone review and if appropriate approve?

Thanks,

Matt

ld/testsuite/ChangeLog:
2010-11-29  Matthew Gretton-Dann  <matthew.gretton-dann@arm.com>

	* ld-scripts/script.exp: Remove full path from REGION_ALIAS test
	names.

-- 
Matthew Gretton-Dann
Principal Engineer - PDSW Tools
ARM Ltd

[-- Attachment #2: 1011-ld-script-region-alias.patch --]
[-- Type: text/x-patch, Size: 751 bytes --]

diff --git a/ld/testsuite/ld-scripts/script.exp b/ld/testsuite/ld-scripts/script.exp
index 2e69922..267c0be 100644
--- a/ld/testsuite/ld-scripts/script.exp
+++ b/ld/testsuite/ld-scripts/script.exp
@@ -130,9 +130,10 @@ if ![ld_simple_link $ld tmpdir/script "$flags -T $srcdir/$subdir/memory.t tmpdir
 set test_script_list [lsort [glob $srcdir/$subdir/region-alias-*.t]]
 
 foreach test_script $test_script_list {
+    set testname [file tail $test_script]
     if ![ld_simple_link $ld tmpdir/script "$flags -T $test_script tmpdir/script.o"] {
-        xfail "REGION_ALIAS: $test_script"
+        xfail "REGION_ALIAS: $testname"
     } else {
-        xpass "REGION_ALIAS: $test_script"
+        xpass "REGION_ALIAS: $testname"
     }
 }

^ permalink raw reply	[flat|nested] 2+ messages in thread

* Re: [PATCH/RFA]: Tidy up names of ld-script REGION_ALIAS tests
  2010-11-29 17:11 [PATCH/RFA]: Tidy up names of ld-script REGION_ALIAS tests Matthew Gretton-Dann
@ 2010-11-30 10:16 ` Alan Modra
  0 siblings, 0 replies; 2+ messages in thread
From: Alan Modra @ 2010-11-30 10:16 UTC (permalink / raw)
  To: Matthew Gretton-Dann; +Cc: binutils

On Mon, Nov 29, 2010 at 01:14:52PM +0000, Matthew Gretton-Dann wrote:
> 	* ld-scripts/script.exp: Remove full path from REGION_ALIAS test
> 	names.

OK.

-- 
Alan Modra
Australia Development Lab, IBM

^ permalink raw reply	[flat|nested] 2+ messages in thread

end of thread, other threads:[~2010-11-30  8:05 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2010-11-29 17:11 [PATCH/RFA]: Tidy up names of ld-script REGION_ALIAS tests Matthew Gretton-Dann
2010-11-30 10:16 ` Alan Modra

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).