[PATCH] Allow gdb version 10 in pretty printer tests.
Stefan Liebler
stli@linux.ibm.com
Thu Mar 11 13:10:54 GMT 2021
On 3/11/21 11:25 AM, Florian Weimer wrote:
> * Stefan Liebler via Libc-alpha:
>
>> With gdb 10, the pretty printer tests are UNSUPPORTED::
>> The gdb version string (gdb -v) is incorrectly formatted.
>>
>> This is observable in:
>> nptl/test-cond-printers, nptl/test-condattr-printers,
>> nptl/test-mutex-printers, nptl/test-mutexattr-printers,
>> nptl/test-rwlock-printers, nptl/test-rwlockattr-printers
>>
>> After updating the regexp and building with debug-info,
>> all those tests are passing.
>> ---
>> scripts/test_printers_common.py | 3 ++-
>> 1 file changed, 2 insertions(+), 1 deletion(-)
>>
>> diff --git a/scripts/test_printers_common.py b/scripts/test_printers_common.py
>> index 0d515891f6..34a3df6e6b 100644
>> --- a/scripts/test_printers_common.py
>> +++ b/scripts/test_printers_common.py
>> @@ -72,7 +72,8 @@ try:
>> # The gdb version string is "GNU gdb <PKGVERSION><version>", where
>> # PKGVERSION can be any text. We assume that there'll always be a space
>> # between PKGVERSION and the version number for the sake of the regexp.
>> - version_match = re.search(r'GNU gdb .* ([1-9]+)\.([0-9]+)', gdb_version_out)
>> + version_match = re.search(r'GNU gdb .* ([1-9][0-9]*)\.([0-9]+)',
>> + gdb_version_out)
>
> Looks okay to me, thanks.
>
> I really value your test debugging skills.
>
> Florian
>
Thanks.
Committed.
More information about the Libc-alpha
mailing list