Re: printers.py issue

[Jonathan Wakely <jwakely@redhat.com>]

On 04/03/21 04:35 +0000, Hoyer, David via Libstdc++ wrote:
>I just realized that some of my original email explaining the issue was removed from the thread

Yes, my fault, sorry.

>Problem 1:  the function find_type was changed such that search now uses typ.unqualified() instead of typ.name.
>In the case of a class, the former prepends class to the search string so that when it runs gdb.lookup_type(search), it does not find the definition.   If I switch it back to typ.name, this function works

But typ.name doesn't exist in GDB 7.6, which is why it's not used. See
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=67440 for the history.

The differences between gdb.Type.name and gdb.Type.tag and
str(gdb.Type) in different versions of GCC and/or GDB  are a non-stop
pain in the arse.

I think typ.unqualified().tag should work consistently. Maybe even
typ.tag but I don't trust it.


>Problem 2:  We are seeing an error in strip_versioned_namespace
>  File "/usr/lib/x86_64-linux-gnu/../../share/gcc-8/python/libstdcxx/v6/printers.py", line 668, in __init__
>      self.typename = strip_versioned_namespace(typename)
>  File "/usr/lib/x86_64-linux-gnu/../../share/gcc-8/python/libstdcxx/v6/printers.py", line 114, in strip_versioned_namespace
>    return typename.replace(_versioned_namespace, '')
>AttributeError: 'gdb.Type' object has no attribute 'replace'
>Error occurred in Python command: 'gdb.Type' object has no attribute 'replace'

This one would be easy to fix by converting to a string first.  But I
wonder why it isn't failing for everybody. All calls to the function
strip_versioned_namespace should be getting passed a string, not a
gdb.Type object.


>
>From: Hoyer, David
>Sent: Friday, February 26, 2021 4:31 PM
>To: Jonathan Wakely <jwakely.gcc@gmail.com>
>Cc: gcc@gcc.gnu.org; libstdc++ <libstdc++@gcc.gnu.org>
>Subject: RE: printers.py issue
>
>Thank you for redirecting to the correct group!
>
>Requested version information:
>GNU gdb (Debian 8.2.1-2+b3) 8.2.1
>Python 3.7.3
>
>From: Jonathan Wakely <jwakely.gcc@gmail.com<mailto:jwakely.gcc@gmail.com>>
>Sent: Friday, February 26, 2021 4:27 PM
>To: Hoyer, David <David.Hoyer@netapp.com<mailto:David.Hoyer@netapp.com>>
>Cc: gcc@gcc.gnu.org<mailto:gcc@gcc.gnu.org>; libstdc++ <libstdc++@gcc.gnu.org<mailto:libstdc++@gcc.gnu.org>>
>Subject: Re: printers.py issue
>
>NetApp Security WARNING: This is an external email. Do not click links or open attachments unless you recognize the sender and know the content is safe.
>
>
>On Fri, 26 Feb 2021, 21:59 Hoyer, David via Gcc, <gcc@gcc.gnu.org<mailto:gcc@gcc.gnu.org>> wrote:
>COLLECT_GCC=gcc
>COLLECT_LTO_WRAPPER=/usr/lib/gcc/x86_64-linux-gnu/8/lto-wrapper
>OFFLOAD_TARGET_NAMES=nvptx-none
>OFFLOAD_TARGET_DEFAULT=1
>Target: x86_64-linux-gnu
>Configured with: ../src/configure -v --with-pkgversion='Debian 8.3.0-6' --with-bugurl=file:///usr/share/doc/gcc-8/README.Bugs --enable-languages=c,ada,c++,go,brig,d,fortran,objc,obj-c++ --prefix=/usr --with-gcc-major-version-only --program-suffix=-8 --program-prefix=x86_64-linux-gnu- --enable-shared --enable-linker-build-id --libexecdir=/usr/lib --without-included-gettext --enable-threads=posix --libdir=/usr/lib --enable-nls --enable-bootstrap --enable-clocale=gnu --enable-libstdcxx-debug --enable-libstdcxx-time=yes --with-default-libstdcxx-abi=new --enable-gnu-unique-object --disable-vtable-verify --enable-libmpx --enable-plugin --enable-default-pie --with-system-zlib --with-target-system-zlib --enable-objc-gc=auto --enable-multiarch --disable-werror --with-arch-32=i686 --with-abi=m64 --with-multilib-list=m32,m64,mx32 --enable-multilib --with-tune=generic --enable-offload-targets=nvptx-none --without-cuda-driver --enable-checking=release --build=x86_64-linux-gnu --host=x86_64-linux-gnu --target=x86_64-linux-gnu
>Thread model: posix
>gcc version 8.3.0 (Debian 8.3.0-6)
>
>We are noticing some problems using GDB with printers.py
>
>This belongs on the libstdc++ list, CC'd.
>
>Which version of GDB are you using, and which version of Python does it use?
>
>
>