Mirror of the gdb-patches mailing list
 help / color / mirror / Atom feed
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])


  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