From mboxrd@z Thu Jan  1 00:00:00 1970
Return-Path: <libc-alpha-return-76296-listarch-libc-alpha=sources.redhat.com@sourceware.org>
Received: (qmail 85241 invoked by alias); 23 Dec 2016 17:40:33 -0000
Mailing-List: contact libc-alpha-help@sourceware.org; run by ezmlm
Precedence: bulk
List-Id: <libc-alpha.sourceware.org>
List-Subscribe: <mailto:libc-alpha-subscribe@sourceware.org>
List-Archive: <http://sourceware.org/ml/libc-alpha/>
List-Post: <mailto:libc-alpha@sourceware.org>
List-Help: <mailto:libc-alpha-help@sourceware.org>, <http://sourceware.org/ml/#faqs>
Sender: libc-alpha-owner@sourceware.org
Received: (qmail 85213 invoked by uid 89); 23 Dec 2016 17:40:32 -0000
Authentication-Results: sourceware.org; auth=none
X-Virus-Found: No
X-Spam-SWARE-Status: No, score=-1.1 required=5.0 tests=AWL,BAYES_00,FREEMAIL_ENVFROM_END_DIGIT,FREEMAIL_FROM,RCVD_IN_DNSWL_NONE,RCVD_IN_SORBS_SPAM,SPF_PASS autolearn=no version=3.3.2 spammy=
X-Spam-User: qpsmtpd, 2 recipients
X-HELO: mail-yw0-f193.google.com
X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed;
        d=1e100.net; s=20161025;
        h=x-gm-message-state:mime-version:in-reply-to:references:from:date
         :message-id:subject:to:cc;
        bh=g4KkUuoGzDASlHAEjk2oMK87DDwUtRRUEV07O7ifhc8=;
        b=EeQwieUbiPGF498qzedvCewEyQZAYQ3hP8sFIBOvQbXPCY8+3N3j01icaA8bgwaLAl
         DFbAEg5SLkMIjEelKHpKDlRS0iTLMR6eCKD3U9WeBfL/0Nua+MU1jwx7hlZLmdq2mxFu
         9FdfV3t9BuyEuWdWskh34xWvkl8JQ6DY2vzrQaIT2Xwk2M1Ojf3uD0RZcsjy3LiCOWmz
         tXn4zQrhz59DIIRZTWjooinwD5THUmRsNTRIY4+sDnDEzOpE+aRANtNWvDl4AAM6ukbF
         SHWAnKwFwpAgDrKqvVuTU1xWGiNPYMJSOlxUcrl9q8XxWDeNOrJpV6yMYZRIRvONebAw
         wr9w==
X-Gm-Message-State: AIkVDXKIbMPENpwEljnhkkdEqw1ywS+rNIamJEGGrsLvK7FViVxIp2u3d7buN1v2XaaBFXilPqUrA7LDATQ+MQ==
X-Received: by 10.13.218.199 with SMTP id c190mr11940878ywe.270.1482514820782;
 Fri, 23 Dec 2016 09:40:20 -0800 (PST)
MIME-Version: 1.0
In-Reply-To: <1482512736.14990.831.camel@redhat.com>
References: <alpine.DEB.2.20.1612081855170.9769@digraph.polyomino.org.uk>
 <1481576088-15304-1-git-send-email-omgalvan.86@gmail.com> <alpine.DEB.2.20.1612141820060.32751@digraph.polyomino.org.uk>
 <CAN19L9GLm7Ey5ni=yrTQ7KioW2qF8rEHinincs85iS_2JKPmRQ@mail.gmail.com>
 <CAN19L9FAbzN96JMV=i=o5LNic6ctRGUnoxYqSe09NvB-ZaTJMQ@mail.gmail.com>
 <cb8ede5a-f9ce-9d3b-9fe9-b6708c1388eb@sourceware.org> <CAN19L9FSEUW+1_0ziESsAd-Sr-Nd9gnk9jdTBPKiNnqmCQ2a8A@mail.gmail.com>
 <1482424494.14990.810.camel@redhat.com> <CAN19L9Feu-=SAiwuTxsb1HNM96RhbrttFG7Q3O_AH=X942jRag@mail.gmail.com>
 <1482447010.14990.815.camel@redhat.com> <CAN19L9GEmszxLiGmUAiMYbiTXjs92jZ=pE3v-3iLwygXRoRE8g@mail.gmail.com>
 <1482512736.14990.831.camel@redhat.com>
From: Martin Galvan <omgalvan.86@gmail.com>
Date: Fri, 23 Dec 2016 17:40:00 -0000
Message-ID: <CAN19L9FYiW7Y4XFxY3uteaV=HYUOkhz6TihBxNyYF9fAfmNngQ@mail.gmail.com>
Subject: Re: [PATCH v11] Add pretty printers for the NPTL lock types
To: Torvald Riegel <triegel@redhat.com>
Cc: siddhesh@sourceware.org, Joseph Myers <joseph@codesourcery.com>, 
	libc-alpha@sourceware.org
Content-Type: text/plain; charset=UTF-8
X-SW-Source: 2016-12/txt/msg00924.txt.bz2

2016-12-23 14:05 GMT-03:00 Torvald Riegel <triegel@redhat.com>:
> After *lots* of poking, it seems that something in the pexpect/gdb
> combination in my case was adding terminal escape sequences to the gdb
> output.  That made the error string look empty when printed, but it was
> not a zero-sized string and thus does not evaluate to false.

That's really weird. In any case, if I understood correctly: you were
using the newly built --with-python gdb, and still got an error saying
that gdb doesn't have Python support? If not, what was the actual
error?

> Switching to gdb 7.12 does not require ncurses or a termcap library
> anymore, and works with the existing pretty printers script.
>
> I suppose raising the minimally required gdb version would be helpful.

Alright, I can do that.

> It would also be helpful if scripts/test_printers_common.py would not
> discard the error when it cannot load the pretty printer modules for
> some reason.

Makes sense, will add it in a future patch.

> And nptl/test-mutex-printers fails in my setup, BTW.

How? What does the .out say? Does it fail for 7.12 too?