In order for GDB to auto-load the pretty printers, they must be installed as "libstdc++.$ext-gdb.py", where 'libstdc++.$ext' is the name of the object file that is loaded by GDB [1], i.e. the libstdc++ shared library.
The approach taken in libstdc++-v3/python/Makefile.am is to loop over files matching 'libstdc++*' in $(DESTDIR)$(toolexeclibdir) and choose the last file matching that glob that is not a symlink, the Libtool '*.la' file or a Python file. That works fine for ELF targets where the matching names are: libstdc++.a libstdc++.so libstdc++.so.6 libstdc++.so.6.0.29 But not for macOS with: libstdc++.6.dylib libstdc++.a Or MinGW with: libstdc++-6.dll libstdc++.dll.a Try to make a better job at installing the pretty printers with the correct name by copying the approach taken by isl [2], that is, using a sed invocation on the the Libtool-generated 'libstdc++.la' to read the correct name for the current platform. [1] https://sourceware.org/gdb/onlinedocs/gdb/objfile_002dgdbdotext-file.html [2] https://repo.or.cz/isl.git/blob/HEAD:/Makefile.am#l611 libstdc++-v3/ PR libstdc++/99453 * python/Makefile.am: Install libstdc++*-gdb.py more robustly * python/Makefile.in: Regenerate Co-authored-by: Jonathan Wakely <jwak...@redhat.com> --- Notes: Hello, this is my first patch to this project. This patch aims to install the GDB Python file for libstdc++ more robustly so that it is automatically loaded on more platforms. I tested that it gets installed with the correct name on Ubuntu (x86_64-pc-linux-gnu) and macOS (x86_64-apple-darwin19.6.0). I did not succeed in building GCC on Windows, I'm very new to this platform. Here are examples of successful builds on these two platforms (you must be logged in on GitHub to see the details): Ubuntu: https://github.com/phil-blain/gcc/runs/2103367197?check_suite_focus=true#step:6:5 macOS: https://github.com/phil-blain/gcc/runs/2103367199?check_suite_focus=true#step:6:5 Fetch-It-Via: git fetch https://github.com/phil-blain/gcc libstdcxx-pretty-printers-install-filename libstdc++-v3/python/Makefile.am | 20 ++++---------------- 1 file changed, 4 insertions(+), 16 deletions(-) diff --git a/libstdc++-v3/python/Makefile.am b/libstdc++-v3/python/Makefile.am index 01517a2a5..0c2b207b8 100644 --- a/libstdc++-v3/python/Makefile.am +++ b/libstdc++-v3/python/Makefile.am @@ -44,21 +44,9 @@ gdb.py: hook.in Makefile install-data-local: gdb.py @$(mkdir_p) $(DESTDIR)$(toolexeclibdir) ## We want to install gdb.py as SOMETHING-gdb.py. SOMETHING is the -## full name of the final library. We want to ignore symlinks, the -## .la file, and any previous -gdb.py file. This is inherently -## fragile, but there does not seem to be a better option, because -## libtool hides the real names from us. - @here=`pwd`; cd $(DESTDIR)$(toolexeclibdir); \ - for file in libstdc++.*; do \ - case $$file in \ - *-gdb.py) ;; \ - *.la) ;; \ - *) if test -h $$file; then \ - continue; \ - fi; \ - libname=$$file;; \ - esac; \ - done; \ - cd $$here; \ +## full name of the final library. We use the libtool .la file to get +## the correct name. + @libname=`sed -ne "/^library_names=/{s/.*='//;s/'$$//;s/ .*//;p;}" \ + $(DESTDIR)$(toolexeclibdir)/libstdc++.la`; \ echo " $(INSTALL_DATA) gdb.py $(DESTDIR)$(toolexeclibdir)/$$libname-gdb.py"; \ $(INSTALL_DATA) gdb.py $(DESTDIR)$(toolexeclibdir)/$$libname-gdb.py base-commit: 6e885ad3287388192e52e9b524dbaa408507c0a4 -- 2.31.1
