From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (qmail 3879 invoked by alias); 16 Jan 2005 19:26:22 -0000 Mailing-List: contact gdb-help@sources.redhat.com; run by ezmlm Precedence: bulk List-Subscribe: List-Archive: List-Post: List-Help: , Sender: gdb-owner@sources.redhat.com Received: (qmail 3603 invoked from network); 16 Jan 2005 19:26:17 -0000 Received: from unknown (HELO sibelius.xs4all.nl) (82.92.89.47) by sourceware.org with SMTP; 16 Jan 2005 19:26:17 -0000 Received: from elgar.sibelius.xs4all.nl (elgar.sibelius.xs4all.nl [192.168.0.2]) by sibelius.xs4all.nl (8.13.0/8.13.0) with ESMTP id j0GJQGcw001951; Sun, 16 Jan 2005 20:26:16 +0100 (CET) Received: from elgar.sibelius.xs4all.nl (localhost [127.0.0.1]) by elgar.sibelius.xs4all.nl (8.12.6p3/8.12.6) with ESMTP id j0GJQGAo005767; Sun, 16 Jan 2005 20:26:16 +0100 (CET) (envelope-from kettenis@elgar.sibelius.xs4all.nl) Received: (from kettenis@localhost) by elgar.sibelius.xs4all.nl (8.12.6p3/8.12.6/Submit) id j0GJQGX7005764; Sun, 16 Jan 2005 20:26:16 +0100 (CET) Date: Sun, 16 Jan 2005 19:26:00 -0000 Message-Id: <200501161926.j0GJQGX7005764@elgar.sibelius.xs4all.nl> From: Mark Kettenis To: gdb-patches@sources.redhat.com CC: gdb@sources.redhat.com Subject: [PATCH/RFA] sim autoconf conversion fallout X-SW-Source: 2005-01/txt/msg00080.txt.bz2 There still a lot of breakage in the sim tree: * Several simulators no longer build because the common directory isn't configured. AFAICT all similators need that directory, so I propose the attached patch. * The --enable-sim option is essentially a no-op now, since all simulators are built unconditionally if gdb is configured for a matching target. Why has the option been retained while all the logic that used it is gone? Why was the logic removed in the first place? * In the past several simulators were only built when compiling with gcc. That code was ripped out. Why? The result of these problems is that gdb cannot be built on many, many systems anymore. After the attached patch has been applied, my immediate concern the ERC32 SPARC V7 simulator. The simulator is currently broken (and has been for quite some time I guess and even before that it only built on i386 and sparc hosts). As a result GDB won't build for any sparc*-*-* target. Is there any interest in retaining this simulator? If so, we should at least but some of the removed logic back. Otherwise, I propose we remove the ERC32 simulator. Mark Index: ChangeLog from Mark Kettenis * configure.ac: Always configure the common subdirectory. Index: configure.ac =================================================================== RCS file: /cvs/src/src/sim/configure.ac,v retrieving revision 1.2 diff -u -p -r1.2 configure.ac --- configure.ac 12 Jan 2005 16:23:11 -0000 1.2 +++ configure.ac 16 Jan 2005 19:25:53 -0000 @@ -33,6 +33,8 @@ yes | no) ;; *) AC_MSG_ERROR(bad value ${enableval} given for --enable-sim option) ;; esac]) +AC_CONFIG_SUBDIRS(common) + # WHEN ADDING ENTRIES TO THIS MATRIX: # Make sure that the left side always has two dashes. Otherwise you @@ -42,13 +44,11 @@ esac]) if test "${enable_sim}" != no; then testsuite=no - common=no igen=no case "${target}" in arm*-*-* | thumb*-*-* | strongarm*-*-* | xscale-*-*) AC_CONFIG_SUBDIRS(arm) testsuite=yes - common=yes ;; d10v-*-*) AC_CONFIG_SUBDIRS(d10v) @@ -56,27 +56,22 @@ if test "${enable_sim}" != no; then frv-*-*) AC_CONFIG_SUBDIRS(frv) testsuite=yes - common=yes ;; h8300*-*-*) AC_CONFIG_SUBDIRS(h8300) testsuite=yes - common=yes ;; m32r-*-*) AC_CONFIG_SUBDIRS(m32r) testsuite=yes - common=yes ;; m68hc11-*-*|m6811-*-*) AC_CONFIG_SUBDIRS(m68hc11) testsuite=yes - common=yes ;; mcore-*-*) AC_CONFIG_SUBDIRS(mcore) testsuite=yes - common=yes ;; mips*-*-*) AC_CONFIG_SUBDIRS(mips) @@ -90,15 +85,12 @@ if test "${enable_sim}" != no; then sh*-*-*) AC_CONFIG_SUBDIRS(sh) testsuite=yes - common=yes ;; powerpc*-*-* ) AC_CONFIG_SUBDIRS(ppc) - common=yes ;; v850*-*-* ) AC_CONFIG_SUBDIRS(v850) - common=yes ;; sparc*-*-*) AC_CONFIG_SUBDIRS(erc32) @@ -107,9 +99,6 @@ if test "${enable_sim}" != no; then if test "$testsuite" = yes; then AC_CONFIG_SUBDIRS(testsuite) fi - if test "$common" = yes; then - AC_CONFIG_SUBDIRS(common) - fi if test "$igen" = yes; then AC_CONFIG_SUBDIRS(igen) fi