This is the mail archive of the
libc-alpha@sourceware.org
mailing list for the glibc project.
Re: [PING][PATCH v9] Add pretty printers for the NPTL lock types
On 11/07/2016 16:00, Siddhesh Poyarekar wrote:
> On Mon, Jul 11, 2016 at 03:16:19PM +0200, Stefan Liebler wrote:
>> On that machine, I have installed python-pexpect-3.1-1.fc20.noarch.
>> I've tried python2-pexpect-4.0.1-4.fc23.noarch on another machine and you
>> are right, the "echo=False" issue does not appear.
>>
>> But I get fails:
>> FAIL: pretty-printers/test-condvar-attributes-pp:
>> Error: Response does not match the expected pattern.
>> Command: print *attr
>> Expected pattern: pthread_condattr_t
>> Response: Python Exception <class 'gdb.error'> No struct type named
>> pthread_condattr.:
>>
>> Python Exception <class 'gdb.error'> No struct type named pthread_condattr.:
>>
>> $3 = {__size = "\000\000\000", __align = 0}
> <snip>
>> The gdb version is gdb-7.10.1-30.fc23.s390x.
>
> That's odd, it looks like the pretty printer is still not registered.
> Have you set up your gdbinit to register the printer?
>
> The packages now look like the same as my setup (since it seems we're
> both on F23), so it seems to be something else.
>
>> Nevertheless is it possible to check the correct versions and warn
>> at configuring or mark the tests as unsupported if pexpect is not installed
>> or you have the false version?
>
> Yes, that should be the correct way to do it. I'm going to revert the
> patch because it is not as isolated as I had thought.
Good, because I was about to ask to revert it.
>
> Martin, so things for you to work on for the next iteration of the
> patch:
>
> 1. Redirect test output to a file like other tests
>
> 2. Eliminate the need to use a .gdbinit because distributions will
> break without it. I should have caught that but I was in too much
> of a hurry to get the patch in :/
>
> 3. Feature checking during configure to determine things like minimum
> required gdb version, python-pexpect version, etc. to make sure
> that tests work correctly.
>
> You can split this into a series of patches, one with what you have
> now (with the output redirection fixed), a second that eliminates
> gdbinit use and a third that adds the configure checks.
>
> Siddhesh
>