From: Tom Tromey <tromey@redhat.com>
To: Andreas Schwab <schwab@linux-m68k.org>
Cc: gdb-patches@sourceware.org
Subject: Re: RFC: remove a bunch of testsuite/**/configure scripts
Date: Tue, 01 Mar 2011 17:15:00 -0000 [thread overview]
Message-ID: <m3k4gi69tu.fsf@fleche.redhat.com> (raw)
In-Reply-To: <m2d3mbn74v.fsf@igel.home> (Andreas Schwab's message of "Mon, 28 Feb 2011 23:09:36 +0100")
Tom> I omitted the removed files from the appended.
Andreas> You omitted the change to configure.ac as well.
Whoops. I've appended that part.
Tom> * configure.ac: Don't AC_CONFIG_HEADER in gdb.hp or gdb.stabs.
Andreas> s/AC_CONFIG_HEADER/AC_CONFIG_SUBDIRS/
Tom> Remove stabs-specific logic. Test for existence of ../gdbtk
Tom> before configuring gdb.gdbtk. Build Makefiles in gdb.hp
Tom> subdirectories and gdb.stabs.
Andreas> + gdb.cell?
Thanks, I updated this part of the entry to read:
* configure.ac: Don't AC_CONFIG_SUBDIRS in gdb.hp or gdb.stabs.
Remove stabs- and cell-specific logic. Test for existence of
../gdbtk before configuring gdb.gdbtk. Build Makefiles in gdb.hp
subdirectories, gdb.cell, and and gdb.stabs.
Tom
diff --git a/gdb/testsuite/configure.ac b/gdb/testsuite/configure.ac
index 9f96f95..8631442 100644
--- a/gdb/testsuite/configure.ac
+++ b/gdb/testsuite/configure.ac
@@ -37,60 +37,10 @@ esac
AM_CONDITIONAL(GMAKE, test "$MAKE_IS_GNU" = yes)
AC_PROG_MAKE_SET
-# Add HP-specific tests when appropriate.
-case $target in
- hppa*-*-hpux*)
- AC_CONFIG_SUBDIRS(gdb.hp) ;;
-esac
-
-# Add Cell/B.E. specific tests when appropriate.
-# We need support for both PowerPC and SPU targets configured in.
-case $target in
- powerpc*-*-linux*)
- have_spu_target=no
- for targ in `echo $enable_targets | sed 's/,/ /g'`
- do
- case "${targ}" in
- spu*|all) have_spu_target=yes ;;
- esac
- done
- if test x$have_spu_target = xyes; then
- AC_CONFIG_SUBDIRS(gdb.cell)
- fi
- ;;
-esac
-
-# With stabs.
-AC_ARG_WITH(stabs,
-[ --with-stabs arrange to use stabs instead of host debug format],,
-[# We enable stabs tests by default on selected targets.
-case $target in
- powerpc-*-aix* \
- | rs6000-*-aix* \
- | *-*-*bsd* \
- | *-*-go32* \
- | *-*-linux* \
- | *-*-lynxos* \
- | *-sun-* \
- | hppa*-*-* \
- | *-*-elf* \
- | *-*-cygwin* \
- | *-*-mingw* \
- )
- with_stabs=yes ;;
- *)
- with_stabs=no ;;
-esac])
-
-# Add stabs tests when appropriate.
-if test $with_stabs = yes; then
- AC_CONFIG_SUBDIRS(gdb.stabs)
-fi
-
# Enable gdbtk.
AC_ARG_ENABLE(gdbtk,
[ --enable-gtk enable gdbtk graphical user interface (GUI)],,
- [if test -d $srcdir/gdb.gdbtk; then
+ [if test -d $srcdir/../gdbtk && test -d $srcdir/gdb.gdbtk; then
enable_gdbtk=yes
else
enable_gdbtk=no
@@ -141,9 +91,12 @@ AC_EXEEXT
AC_OUTPUT([Makefile \
gdb.ada/Makefile \
gdb.arch/Makefile gdb.asm/Makefile gdb.base/Makefile \
- gdb.cp/Makefile gdb.disasm/Makefile gdb.dwarf2/Makefile \
+ gdb.cell/Makefile gdb.cp/Makefile gdb.disasm/Makefile gdb.dwarf2/Makefile \
gdb.fortran/Makefile gdb.server/Makefile gdb.java/Makefile \
+ gdb.hp/Makefile gdb.hp/gdb.objdbg/Makefile gdb.hp/gdb.base-hp/Makefile \
+ gdb.hp/gdb.aCC/Makefile gdb.hp/gdb.compat/Makefile \
+ gdb.hp/gdb.defects/Makefile \
gdb.mi/Makefile gdb.modula2/Makefile gdb.multi/Makefile \
gdb.objc/Makefile gdb.opencl/Makefile gdb.opt/Makefile gdb.pascal/Makefile \
- gdb.python/Makefile gdb.reverse/Makefile \
+ gdb.python/Makefile gdb.reverse/Makefile gdb.stabs/Makefile \
gdb.threads/Makefile gdb.trace/Makefile gdb.xml/Makefile])
next prev parent reply other threads:[~2011-03-01 17:15 UTC|newest]
Thread overview: 6+ messages / expand[flat|nested] mbox.gz Atom feed top
2011-02-28 22:03 Tom Tromey
2011-02-28 22:11 ` Andreas Schwab
2011-03-01 17:15 ` Tom Tromey [this message]
2011-03-01 4:53 ` Joel Brobecker
2011-03-01 17:20 ` Tom Tromey
2011-03-03 16:57 ` Tom Tromey
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=m3k4gi69tu.fsf@fleche.redhat.com \
--to=tromey@redhat.com \
--cc=gdb-patches@sourceware.org \
--cc=schwab@linux-m68k.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox