Patchwork [v3] RFA: PATCH to abi_check FAIL message

login
register
mail settings
Submitter Jason Merrill
Date Sept. 6, 2010, 5:07 a.m.
Message ID <4C8476FF.108@redhat.com>
Download mbox | patch
Permalink /patch/63875/
State New
Headers show

Comments

Jason Merrill - Sept. 6, 2010, 5:07 a.m.
I find that sometimes I don't notice the

FAIL: abi_check

line because it doesn't have a subdirectory, so it looks like a Java 
test.  This patch changes it to

FAIL: libstdc++-abi/abi_check

OK for trunk?
Paolo Carlini - Sept. 6, 2010, 9:23 a.m.
On 09/06/2010 07:07 AM, Jason Merrill wrote:
> I find that sometimes I don't notice the
>
> FAIL: abi_check
>
> line because it doesn't have a subdirectory, so it looks like a Java
> test.  This patch changes it to
>
> FAIL: libstdc++-abi/abi_check
>
> OK for trunk?
OK, thanks.

Paolo.

Patch

commit 8dd23936e17335f82940d43c5d286a52234da3e2
Author: Jason Merrill <jason@redhat.com>
Date:   Mon Sep 6 00:10:21 2010 -0400

    	* testsuite/libstdc++-abi/abi.exp: Add "libstdc++-abi/" to
    	abi_check FAIL message.

diff --git a/libstdc++-v3/testsuite/libstdc++-abi/abi.exp b/libstdc++-v3/testsuite/libstdc++-abi/abi.exp
index fd892a0..a0e8f28 100644
--- a/libstdc++-v3/testsuite/libstdc++-abi/abi.exp
+++ b/libstdc++-v3/testsuite/libstdc++-abi/abi.exp
@@ -61,4 +61,4 @@  remote_download "target" "current_symbols.txt" "current_symbols.txt"
 set result [${tool}_load "./abi_check" \
 		[list "--check-verbose" "current_symbols.txt" \
 		     "baseline_symbols.txt"]]
-[lindex $result 0] "abi_check"
+[lindex $result 0] "libstdc++-abi/abi_check"