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

[Andrew Cagney <ac131313 at cygnus dot com>]

Franz Sirl wrote:

> >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.

Just a BTW, 33 failures is pretty good.

> 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

FYI, That isn't a GNU/Linux/PPC specific problem - an external interface
was tweeked.

> 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

Again, not GNU/Linux/PPC specific.  I think that should be split out
into a separate install target.  A new CONFIG_INSTALL variable would
then hook it in. (making any sense?)

Thanks for these,

	enjoy,
		Andrew