From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (qmail 5656 invoked by alias); 9 Aug 2004 23:12:16 -0000 Mailing-List: contact gdb-patches-help@sources.redhat.com; run by ezmlm Precedence: bulk List-Subscribe: List-Archive: List-Post: List-Help: , Sender: gdb-patches-owner@sources.redhat.com Received: (qmail 5639 invoked from network); 9 Aug 2004 23:12:13 -0000 Received: from unknown (HELO mx1.redhat.com) (66.187.233.31) by sourceware.org with SMTP; 9 Aug 2004 23:12:13 -0000 Received: from int-mx1.corp.redhat.com (int-mx1.corp.redhat.com [172.16.52.254]) by mx1.redhat.com (8.12.10/8.12.10) with ESMTP id i79NCDe1032145; Mon, 9 Aug 2004 19:12:13 -0400 Received: from pobox.corp.redhat.com (pobox.corp.redhat.com [172.16.52.156]) by int-mx1.corp.redhat.com (8.11.6/8.11.6) with ESMTP id i79NCCa26865; Mon, 9 Aug 2004 19:12:12 -0400 Received: from [172.16.50.10] (vpn50-10.rdu.redhat.com [172.16.50.10]) by pobox.corp.redhat.com (8.12.8/8.12.8) with ESMTP id i79NCBgU017641; Mon, 9 Aug 2004 19:12:12 -0400 Subject: Re: [PATCH] Remove trailing references to tix From: "Martin M. Hunt" To: Nathanael Nerode Cc: gdb-patches@sources.redhat.com, insight@sources.redhat.com In-Reply-To: <20040809033526.GA18783@twcny.rr.com> References: <20040809033526.GA18783@twcny.rr.com> Content-Type: text/plain Organization: Red Hat Inc. Message-Id: <1092093126.2945.6.camel@dragon> Mime-Version: 1.0 Date: Mon, 09 Aug 2004 23:12:00 -0000 Content-Transfer-Encoding: 7bit X-SW-Source: 2004-08/txt/msg00345.txt.bz2 Last time we discussed this was over a year ago and we agreed (reluctantly) to wait a bit. mostly because there was no compelling case for removing it. I certainly have no intention of ever putting it back in Insight. Its been out of use for almost 2 years. I haven't heard of anyone else even considering using it, and the tix in cvs is very, very old. So I vote "yes". Any disagreement? On Sun, 2004-08-08 at 20:35, Nathanael Nerode wrote: > I noticed that there are a lot of references to tix around -- even though > it's been completely wiped from HEAD and isn't used anymore. > > This pokes src-release, so it's not covered under the ALL rule. > > Tested src-release by building an insight tarball with src-release. Insight > doesn't currently use tix at all, quite clearly, so I suspect the rest > is obvious. > > Note that this is all top-level stuff; I'm actually doing it as a matter of > general maintenance, not insight-specific stuff. > > OK to apply? Please reply to one of the lists, as my mail is flaky of late. > > * Makefile.def, configure.in, src-release: Remove useless, bogus > references to tix. > * Makefile.in, configure: Regenerate. > > --- Makefile.def 2004-08-08 23:12:15.000000000 -0400 > +++ Makefile.def.tixless 2004-08-08 23:11:56.000000000 -0400 > @@ -98,7 +98,6 @@ > host_modules= { module= expect; with_x=true; }; > host_modules= { module= guile; with_x=true; }; > host_modules= { module= tk; with_x=true; }; > -host_modules= { module= tix; with_x=true; }; > host_modules= { module= libtermcap; no_check=true; > missing=mostlyclean; > missing=clean; > @@ -275,7 +274,6 @@ > dependencies = { module=install-gdb; on=install-tcl; }; > dependencies = { module=install-gdb; on=install-tk; }; > dependencies = { module=install-gdb; on=install-itcl; }; > -dependencies = { module=install-gdb; on=install-tix; }; > dependencies = { module=install-gdb; on=install-libgui; }; > > dependencies = { module=configure-libgui; on=configure-tcl; }; > @@ -343,10 +341,6 @@ > > dependencies = { module=configure-tk; on=configure-tcl; }; > dependencies = { module=all-tk; on=all-tcl; }; > -dependencies = { module=configure-tix; on=configure-tcl; }; > -dependencies = { module=configure-tix; on=configure-tk; }; > -dependencies = { module=all-tix; on=all-tcl; }; > -dependencies = { module=all-tix; on=all-tk; }; > > dependencies = { module=all-sid; on=all-libiberty; }; > dependencies = { module=all-sid; on=all-bfd; }; > --- src-release 2004-08-08 22:50:15.000000000 -0400 > +++ src-release.tixless 2004-08-08 23:11:06.000000000 -0400 > @@ -309,7 +309,7 @@ > SUPPORT_FILES="$(GDBD_SUPPORT_DIRS)" > > .PHONY: insight.tar.bz2 > -INSIGHT_SUPPORT_DIRS= $(GDB_SUPPORT_DIRS) tcl tk itcl tix libgui > +INSIGHT_SUPPORT_DIRS= $(GDB_SUPPORT_DIRS) tcl tk itcl libgui > insight.tar.bz2: $(DIST_SUPPORT) $(GDB_SUPPORT_DIRS) gdb > $(MAKE) -f $(SELF) gdb-taz TOOL=gdb PACKAGE=insight \ > MD5PROG="$(MD5PROG)" \ > --- configure.in 2004-08-05 19:30:42.000000000 -0400 > +++ configure.in.tixless 2004-08-08 23:15:40.000000000 -0400 > @@ -136,7 +136,7 @@ > > # these libraries are used by various programs built for the host environment > # > -host_libs="intl mmalloc libiberty opcodes bfd readline tcl tk itcl tix libgui zlib libbanshee libcpp" > +host_libs="intl mmalloc libiberty opcodes bfd readline tcl tk itcl libgui zlib libbanshee libcpp" > > # these tools are built for the host environment > # Note, the powerpc-eabi build depends on sim occurring before gdb in order to > @@ -232,7 +232,7 @@ > case ${with_x} in > yes | "") ;; # the default value for this tree is that X11 is available > no) > - skipdirs="${skipdirs} tk tix itcl libgui" > + skipdirs="${skipdirs} tk itcl libgui" > # We won't be able to build gdbtk without X. > enable_gdbtk=no > ;; > @@ -283,17 +283,17 @@ > noconfigdirs="$noconfigdirs byacc" > ;; > i[[3456789]]86-*-vsta) > - noconfigdirs="$noconfigdirs tcl expect dejagnu make texinfo bison patch flex byacc send-pr gprof uudecode dejagnu diff guile perl itcl tix gnuserv gettext" > + noconfigdirs="$noconfigdirs tcl expect dejagnu make texinfo bison patch flex byacc send-pr gprof uudecode dejagnu diff guile perl itcl gnuserv gettext" > ;; > i[[3456789]]86-*-go32* | i[[3456789]]86-*-msdosdjgpp*) > - noconfigdirs="$noconfigdirs tcl tk expect dejagnu send-pr uudecode guile itcl tix gnuserv libffi" > + noconfigdirs="$noconfigdirs tcl tk expect dejagnu send-pr uudecode guile itcl gnuserv libffi" > ;; > i[[3456789]]86-*-mingw32*) > - # noconfigdirs="tcl tk expect dejagnu make texinfo bison patch flex byacc send-pr uudecode dejagnu diff guile perl itcl tix gnuserv" > + # noconfigdirs="tcl tk expect dejagnu make texinfo bison patch flex byacc send-pr uudecode dejagnu diff guile perl itcl gnuserv" > noconfigdirs="$noconfigdirs expect dejagnu autoconf automake send-pr rcs guile perl texinfo libtool" > ;; > i[[3456789]]86-*-beos*) > - noconfigdirs="$noconfigdirs tk itcl tix libgui gdb" > + noconfigdirs="$noconfigdirs tk itcl libgui gdb" > ;; > *-*-cygwin*) > noconfigdirs="$noconfigdirs autoconf automake send-pr rcs guile perl" > @@ -302,13 +302,13 @@ > noconfigdirs="$noconfigdirs rcs" > ;; > ppc*-*-pe) > - noconfigdirs="$noconfigdirs patch diff make tk tcl expect dejagnu autoconf automake texinfo bison send-pr gprof rcs guile perl itcl tix gnuserv" > + noconfigdirs="$noconfigdirs patch diff make tk tcl expect dejagnu autoconf automake texinfo bison send-pr gprof rcs guile perl itcl gnuserv" > ;; > powerpc-*-beos*) > - noconfigdirs="$noconfigdirs tk itcl tix libgui gdb dejagnu readline" > + noconfigdirs="$noconfigdirs tk itcl libgui gdb dejagnu readline" > ;; > *-*-darwin*) > - noconfigdirs="$noconfigdirs tk itcl tix libgui" > + noconfigdirs="$noconfigdirs tk itcl libgui" > ;; > esac > > @@ -462,7 +462,7 @@ > noconfigdirs="$noconfigdirs ${libgcj}" > noconfigdirs="$noconfigdirs target-examples" > noconfigdirs="$noconfigdirs target-libiberty texinfo send-pr" > - noconfigdirs="$noconfigdirs tcl tix tk itcl libgui sim" > + noconfigdirs="$noconfigdirs tcl tk itcl libgui sim" > noconfigdirs="$noconfigdirs expect dejagnu" > # the C++ libraries don't build on top of CE's C libraries > noconfigdirs="$noconfigdirs target-libstdc++-v3" > @@ -563,11 +563,11 @@ > ;; > ia64*-*-elf*) > # No gdb support yet. > - noconfigdirs="$noconfigdirs tix readline mmalloc libgui itcl gdb" > + noconfigdirs="$noconfigdirs readline mmalloc libgui itcl gdb" > ;; > ia64*-**-hpux*) > # No gdb or ld support yet. > - noconfigdirs="$noconfigdirs ${libgcj} tix readline mmalloc libgui itcl gdb ld" > + noconfigdirs="$noconfigdirs ${libgcj} readline mmalloc libgui itcl gdb ld" > ;; > i[[3456789]]86-*-coff | i[[3456789]]86-*-elf) > noconfigdirs="$noconfigdirs ${libgcj}" > @@ -598,7 +598,7 @@ > # Can't build gdb for mingw32 if not native. > case "${host}" in > i[[3456789]]86-*-mingw32) ;; # keep gdb tcl tk expect etc. > - *) noconfigdirs="$noconfigdirs gdb tcl tk expect itcl tix gnuserv" > + *) noconfigdirs="$noconfigdirs gdb tcl tk expect itcl gnuserv" > ;; > esac > ;; > @@ -616,7 +616,7 @@ > # Can't build gdb for Cygwin if not native. > case "${host}" in > *-*-cygwin*) ;; # keep gdb tcl tk expect etc. > - *) noconfigdirs="$noconfigdirs gdb tcl tk expect itcl tix libgui gnuserv" > + *) noconfigdirs="$noconfigdirs gdb tcl tk expect itcl libgui gnuserv" > ;; > esac > ;; > @@ -672,13 +672,13 @@ > ;; > powerpc*-*-winnt* | powerpc*-*-pe* | ppc*-*-pe) > target_configdirs="$target_configdirs target-winsup" > - noconfigdirs="$noconfigdirs gdb tcl tk make expect target-libgloss itcl tix gnuserv ${libgcj}" > + noconfigdirs="$noconfigdirs gdb tcl tk make expect target-libgloss itcl gnuserv ${libgcj}" > # always build newlib. > skipdirs=`echo " ${skipdirs} " | sed -e 's/ target-newlib / /'` > ;; > # This is temporary until we can link against shared libraries > powerpcle-*-solaris*) > - noconfigdirs="$noconfigdirs gdb sim make tcl tk expect itcl tix gnuserv ${libgcj}" > + noconfigdirs="$noconfigdirs gdb sim make tcl tk expect itcl gnuserv ${libgcj}" > ;; > powerpc-*-beos*) > noconfigdirs="$noconfigdirs gdb target-newlib target-libgloss ${libgcj}" > @@ -1636,7 +1636,7 @@ > # leave out the maybe dependencies when enable_gdbtk is false. I'm not > # 100% sure that that's safe though. > > -gdb_tk="maybe-all-tcl maybe-all-tk maybe-all-itcl maybe-all-tix maybe-all-libgui" > +gdb_tk="maybe-all-tcl maybe-all-tk maybe-all-itcl maybe-all-libgui" > case "$enable_gdbtk" in > no) > GDB_TK="" ;; > > -- > There are none so blind as those who will not see. -- Martin M. Hunt Red Hat Inc.