From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (qmail 9563 invoked by alias); 23 Oct 2015 20:24:24 -0000 Mailing-List: contact gcc-patches-help@gcc.gnu.org; run by ezmlm Precedence: bulk List-Id: List-Archive: List-Post: List-Help: Sender: gcc-patches-owner@gcc.gnu.org Received: (qmail 9419 invoked by uid 89); 23 Oct 2015 20:24:23 -0000 Authentication-Results: sourceware.org; auth=none X-Virus-Found: No X-Spam-SWARE-Status: No, score=-1.8 required=5.0 tests=AWL,BAYES_00,RP_MATCHES_RCVD,SPF_HELO_PASS autolearn=ham version=3.3.2 X-HELO: mx1.redhat.com Received: from mx1.redhat.com (HELO mx1.redhat.com) (209.132.183.28) by sourceware.org (qpsmtpd/0.93/v0.84-503-g423c35a) with (AES256-GCM-SHA384 encrypted) ESMTPS; Fri, 23 Oct 2015 20:24:22 +0000 Received: from int-mx10.intmail.prod.int.phx2.redhat.com (int-mx10.intmail.prod.int.phx2.redhat.com [10.5.11.23]) by mx1.redhat.com (Postfix) with ESMTPS id A6FFC54 for ; Fri, 23 Oct 2015 20:24:21 +0000 (UTC) Received: from c64.redhat.com (vpn-230-149.phx2.redhat.com [10.3.230.149]) by int-mx10.intmail.prod.int.phx2.redhat.com (8.14.4/8.14.4) with ESMTP id t9NKOJgS007382; Fri, 23 Oct 2015 16:24:21 -0400 From: David Malcolm To: gcc-patches@gcc.gnu.org Cc: David Malcolm Subject: [PATCH 03/10] libstdc++v3: Explicitly disable carets and colorization within testsuite Date: Fri, 23 Oct 2015 20:24:00 -0000 Message-Id: <1445632918-29617-4-git-send-email-dmalcolm@redhat.com> In-Reply-To: <1445632918-29617-1-git-send-email-dmalcolm@redhat.com> References: <1442957171-22904-1-git-send-email-dmalcolm@redhat.com> <1445632918-29617-1-git-send-email-dmalcolm@redhat.com> X-IsSubscribed: yes X-SW-Source: 2015-10/txt/msg02535.txt.bz2 Later on in this patch kit, with token range underlining, the libstdc++v3 testsuite starts showing numerous failures of the form: FAIL: 17_intro/using_namespace_std_tr1_neg.cc (test for excess errors) The excess errors turn out to be the source code and caret/underlines emitted after an "error": using namespace std::tr1; // { dg-error "is not a namespace-name" } ^~~ However, looking at the results of a control build of r228618, I see the testsuite emit code and carets (albeit without underlines): using namespace std::tr1; // { dg-error "is not a namespace-name" } ^ and for some reason this is treated by dg.exp as: PASS: 17_intro/using_namespace_std_tr1_neg.cc (test for excess errors) It's not clear to me why the status quo isn't treating the lines of dumped source code and caret as "excess errors", but the attached patch explicitly disables carets and colorization. libstdc++-v3/ChangeLog: * testsuite/lib/libstdc++.exp (v3_target_compile): Add -fno-diagnostics-show-caret -fdiagnostics-color=never to option's additional_flags. --- libstdc++-v3/testsuite/lib/libstdc++.exp | 2 ++ 1 file changed, 2 insertions(+) diff --git a/libstdc++-v3/testsuite/lib/libstdc++.exp b/libstdc++-v3/testsuite/lib/libstdc++.exp index 88738b7..ac3654b 100644 --- a/libstdc++-v3/testsuite/lib/libstdc++.exp +++ b/libstdc++-v3/testsuite/lib/libstdc++.exp @@ -462,6 +462,8 @@ proc v3_target_compile { source dest type options } { global STATIC_LIBCXXFLAGS global tool + lappend options "additional_flags=-fno-diagnostics-show-caret -fdiagnostics-color=never" + if { [target_info needs_status_wrapper] != "" && [info exists gluefile] } { lappend options "libs=${gluefile}" lappend options "ldflags=${wrap_flags}" -- 1.8.5.3