From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (qmail 18798 invoked by alias); 21 Dec 2011 12:59:47 -0000 Received: (qmail 18765 invoked by uid 22791); 21 Dec 2011 12:59:46 -0000 X-SWARE-Spam-Status: No, hits=-2.0 required=5.0 tests=AWL,BAYES_00 X-Spam-Check-By: sourceware.org Received: from rock.gnat.com (HELO rock.gnat.com) (205.232.38.15) by sourceware.org (qpsmtpd/0.43rc1) with ESMTP; Wed, 21 Dec 2011 12:59:32 +0000 Received: from localhost (localhost.localdomain [127.0.0.1]) by filtered-rock.gnat.com (Postfix) with ESMTP id C065F2BAD20; Wed, 21 Dec 2011 07:59:31 -0500 (EST) Received: from rock.gnat.com ([127.0.0.1]) by localhost (rock.gnat.com [127.0.0.1]) (amavisd-new, port 10024) with LMTP id HfDdd7JnF0KP; Wed, 21 Dec 2011 07:59:31 -0500 (EST) Received: from joel.gnat.com (localhost.localdomain [127.0.0.1]) by rock.gnat.com (Postfix) with ESMTP id 41F9F2BAD15; Wed, 21 Dec 2011 07:59:31 -0500 (EST) Received: by joel.gnat.com (Postfix, from userid 1000) id A40E1145615; Wed, 21 Dec 2011 04:59:19 -0800 (PST) Date: Wed, 21 Dec 2011 13:12:00 -0000 From: Joel Brobecker To: Jan Kratochvil Cc: gdb-patches@sourceware.org, Sergio Durigan Subject: Re: [patch 2/2] Do not build libgdb.a by default Message-ID: <20111221125919.GJ23376@adacore.com> References: <20111221114720.GC25652@host2.jankratochvil.net> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20111221114720.GC25652@host2.jankratochvil.net> User-Agent: Mutt/1.5.20 (2009-06-14) Mailing-List: contact gdb-patches-help@sourceware.org; run by ezmlm Precedence: bulk List-Id: List-Subscribe: List-Archive: List-Post: List-Help: , Sender: gdb-patches-owner@sourceware.org X-SW-Source: 2011-12/txt/msg00721.txt.bz2 > with the default build of only "gdb" after [patch 1/2] it no longer > makes much sense to build *.o -> libgdb.a and then libgdb.a -> gdb. > Use *.o files directly for gdb. libgdb.a build is still supported for > its rare users, it is just not built by the default rule. Just wondering, who are the rare users? insight? > gdb/ > 2011-12-21 Jan Kratochvil > > Build gdb directly from *.o files not using libgdb.a. > * Makefile.in (SUBDIR_TUI_OBS): Remove duplicate tui.o. > (COMMON_OBS): Remove solib-target.o. > (LIBGDB_OBS, libgdb.a): Move it before the gdb$(EXEEXT) rule. > (gdb$(EXEEXT)): Replace libgdb.a with $(LIBGDB_OBS). > (LIBGDB_OBS, libgdb.a): Move it above. > * configure.tgt (alpha*-*-linux*, alpha*-*-freebsd*) > (alpha*-*-kfreebsd*-gnu, alpha*-*-netbsd*, alpha*-*-knetbsd*-gnu) > (alpha*-*-openbsd*, am33_2.0*-*-linux*, arm*-wince-pe) > (arm*-*-mingw32ce*, arm*-*-linux*, arm*-*-netbsd*, arm*-*-knetbsd*-gnu) > (arm*-*-openbsd*, cris*, frv-*-*, hppa*-*-hpux*, hppa*-*-linux*) > (hppa*-*-netbsd*, hppa*-*-openbsd*, i[34567]86-*-darwin*) > (i[34567]86-*-dicos*, i[34567]86-*-freebsd*, i[34567]86-*-kfreebsd*-gnu) > (i[34567]86-*-netbsd*, i[34567]86-*-knetbsd*-gnu, i[34567]86-*-openbsd*) > (i[34567]86-*-nto*, i[34567]86-*-solaris2.1[0-9]*) > (x86_64-*-solaris2.1[0-9]*, i[34567]86-*-solaris*, i[34567]86-*-linux*) > (i[34567]86-*-gnu*, ia64-*-linux*, m32r*-*-linux*, m68*-*-linux*) > (m68*-*-netbsd*, m68*-*-knetbsd*-gnu, m68*-*-openbsd*) > (microblaze*-linux-*, microblaze*-*-linux*, mips*-sgi-irix5*) > (mips*-sgi-irix6*, mips*-*-linux*, mips*-*-netbsd*) > (mips*-*-knetbsd*-gnu, mips64*-*-openbsd*, powerpc-*-netbsd*) > (powerpc-*-knetbsd*-gnu, powerpc-*-openbsd*, powerpc-*-aix*, rs6000-*-*) > (powerpc-*-linux*, powerpc64-*-linux*, powerpc*-*-*, s390*-*-*) > (sh*-*-linux*, sh*-*-netbsdelf*, sh*-*-knetbsd*-gnu, sh*-*-openbsd*) > (sparc-*-linux*, sparc64-*-linux*, sparc*-*-freebsd*) > (sparc*-*-kfreebsd*-gnu, sparc-*-netbsd*, sparc-*-knetbsd*-gnu) > (sparc64-*-netbsd*, sparc64-*-knetbsd*-gnu, sparc-*-openbsd*) > (sparc64-*-openbsd*, sparc-*-solaris2.[0-6], sparc-*-solaris2.[0-6].*) > (sparc-*-solaris2*, sparcv9-*-solaris2*, sparc64-*-solaris2*) > (vax-*-netbsd*, vax-*-knetbsd*-gnu, x86_64-*-darwin*, x86_64-*-dicos*) > (x86_64-*-linux*, x86_64-*-freebsd*, x86_64-*-kfreebsd*-gnu) > (x86_64-*-netbsd*, x86_64-*-knetbsd*-gnu, x86_64-*-openbsd*) > (xtensa*-*-linux*, xtensa*): Remove solib.o from gdb_target_obs. I support this change. The patch looks good to me also. -- Joel