Re: Patches for GNU/Linux PPC native now in CVS

[Franz Sirl <Franz dot Sirl-kernel at lauterbach dot com>]

Am Die, 22 Feb 2000 schrieb Kevin Buettner:
>Howdy,
>
>I've just committed the changes needed for a native GNU/Linux gdb port.

Great! Thanks for all your work!

>I'm seeing 33 failures in the test suite at the present time.  There
>are some things which *should* work, like backtracing through signal
>handlers, but which don't for some reason.  In the coming days, I
>will attempt to reduce the number of failures.  In particular, I
>will make sure we can backtrace through signal handlers.  (I put
>a lot of time into this in the initial port, and want to make sure
>that it'll work again in the present port.)
>
>I invite those of you who have Linux/PPC machines to try out these
>changes and let me know how they work.  Also, please let me know about
>any problems that you find.

I've just uploaded an RPM with the current CVS source to
ftp://devel.linuxppc.org/users/fsirl/ so people can easily test it.

This is the only warning that shines up without any -W option:

gcc -c -O2 -fsigned-char    -I. -I. -I./config -DHAVE_CONFIG_H -I./../include/opcode -I./../readline/.. -I../bfd -I./../bfd  -I./../include -I../intl -I./../int
l -I./tui    gdbarch.c
gdbarch.c: In function `_initialize_gdbarch':
gdbarch.c:3274: warning: assignment from incompatible pointer type

I configured with --prefix=/usr --disable-gdbtk --disable-sim. disable-gdbtk
seems to be needed to let configure succeed with an already installed
tcltk-8.0.5. I wonder if the toplevel configure.in should be modified to
disable the simulator by default on powerpc-*-linux*?

Then I needed a small patch for gdb/Makefile.in to let "make install" succeed:

--- gdb-5.0/gdb/Makefile.in~    Tue Feb 22 02:35:42 2000
+++ gdb-5.0/gdb/Makefile.in     Tue Feb 22 15:23:21 2000
@@ -649,18 +649,6 @@ install-only:
                fi ; \
                $(INSTALL_PROGRAM) gdb$(EXEEXT) $(bindir)/$$transformed_name$(EXEEXT) ; \
                $(INSTALL_DATA) $(srcdir)/gdb.1 $(man1dir)/$$transformed_name.1
-       $(SHELL) $(srcdir)/../mkinstalldirs $(datadir)/gdbtcl ; \
-       $(SHELL) $(srcdir)/../mkinstalldirs \
-               $(datadir)/gdbtcl/images \
-               $(datadir)/gdbtcl/images2 ; \
-       $(SHELL) $(srcdir)/../mkinstalldirs $(datadir)/gdbtcl/help \
-               $(datadir)/gdbtcl/help/images \
-               $(datadir)/gdbtcl/help/trace ; \
-       cd $(srcdir)/gdbtk/library ; \
-       for i in *.tcl *.ith *.itb images/*.gif images2/*.gif images/icons.txt images2/icons.txt tclIndex help/*.html  help/trace/*.html help/trace/index.toc he
lp/images/*.gif; \
-         do \
-               $(INSTALL_DATA) $$i $(datadir)/gdbtcl/$$i ; \
-         done ;
        @$(MAKE) DO=install "DODIRS=$(SUBDIRS)" $(FLAGS_TO_PASS) subdir_do

 uninstall: force


This should probably be conditionalized on ENABLE_GDBTK.

A quick test of gdb looked very promising, I already installed it as the
default gdb on our build system. What about "info float"? Are you going to
tackle this yourself later or do you want to leave the implementation to
somebody else?

Thanks again,
Franz.