[toplevel patch] delete references to tix

[toplevel patch] delete references to tix

[Nathanael Nerode]

Insight ripped tix out of CVS.  Time to get rid of it I think.  OK?

	* Makefile.def: Remove references to tix.
	* Makefile.tpl: Remove references to tix.
	* Makefile.in: Regenerate.
	* configure.in: Remove references to tix.
	* configure: Regenerate.

Index: Makefile.def
===================================================================
RCS file: /cvs/src/src/Makefile.def,v
retrieving revision 1.11
diff -u -r1.11 Makefile.def
--- Makefile.def	6 Feb 2003 20:36:19 -0000	1.11
+++ Makefile.def	10 Mar 2003 18:16:56 -0000
@@ -64,7 +64,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; no_clean=true; };
 host_modules= { module= utils; no_check=true; };
 
Index: Makefile.tpl
===================================================================
RCS file: /cvs/src/src/Makefile.tpl,v
retrieving revision 1.52
diff -u -r1.52 Makefile.tpl
--- Makefile.tpl	5 Mar 2003 01:50:52 -0000	1.52
+++ Makefile.tpl	10 Mar 2003 18:17:01 -0000
@@ -1320,7 +1320,7 @@
 configure-gdb: maybe-configure-tcl maybe-configure-tk maybe-configure-sim
 GDB_TK = @GDB_TK@
 all-gdb: maybe-all-libiberty maybe-all-opcodes maybe-all-bfd maybe-all-mmalloc maybe-all-readline maybe-all-bison maybe-all-byacc maybe-all-sim $(gdbnlmrequirements) $(GDB_TK)
-install-gdb: maybe-install-tcl maybe-install-tk maybe-install-itcl maybe-install-tix maybe-install-libgui
+install-gdb: maybe-install-tcl maybe-install-tk maybe-install-itcl maybe-install-libgui
 configure-libgui: maybe-configure-tcl maybe-configure-tk
 all-libgui: maybe-all-tcl maybe-all-tk maybe-all-itcl
 
@@ -1351,8 +1351,6 @@
 all-sim: maybe-all-libiberty maybe-all-bfd maybe-all-opcodes maybe-all-readline maybe-configure-gdb
 configure-tk: maybe-configure-tcl
 all-tk: maybe-all-tcl
-configure-tix: maybe-configure-tcl maybe-configure-tk
-all-tix: maybe-all-tcl maybe-all-tk
 all-texinfo: maybe-all-libiberty
 
 # Other host modules.  Warning, these are not well tested.
Index: configure.in
===================================================================
RCS file: /cvs/src/src/configure.in,v
retrieving revision 1.157
diff -u -r1.157 configure.in
--- configure.in	9 Mar 2003 18:43:06 -0000	1.157
+++ configure.in	10 Mar 2003 18:17:12 -0000
@@ -117,7 +117,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"
+host_libs="intl mmalloc libiberty opcodes bfd readline tcl tk itcl libgui zlib"
 
 # these tools are built for the host environment
 # Note, the powerpc-eabi build depends on sim occurring before gdb in order to
@@ -229,7 +229,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 
     ;;
@@ -279,17 +279,17 @@
     noconfigdirs="$noconfigdirs byacc"
     ;;
   i[[3456]]86-*-vsta)
-    noconfigdirs="tcl expect dejagnu make texinfo bison patch flex byacc send-pr gprof uudecode dejagnu diff guile perl itcl tix gnuserv gettext"
+    noconfigdirs="tcl expect dejagnu make texinfo bison patch flex byacc send-pr gprof uudecode dejagnu diff guile perl itcl gnuserv gettext"
     ;;
   i[[3456]]86-*-go32* | i[[3456]]86-*-msdosdjgpp*)
-    noconfigdirs="tcl tk expect dejagnu send-pr uudecode guile itcl tix gnuserv libffi"
+    noconfigdirs="tcl tk expect dejagnu send-pr uudecode guile itcl gnuserv libffi"
     ;;
   i[[3456]]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="expect dejagnu autoconf automake send-pr rcs guile perl texinfo libtool"
     ;;
   i[[3456]]86-*-beos*)
-    noconfigdirs="$noconfigdirs tk itcl tix libgui gdb"
+    noconfigdirs="$noconfigdirs tk itcl libgui gdb"
     ;;
   *-*-cygwin*)
     noconfigdirs="autoconf automake send-pr rcs guile perl"
@@ -298,13 +298,13 @@
     noconfigdirs="rcs"
     ;;
   ppc*-*-pe)
-    noconfigdirs="patch diff make tk tcl expect dejagnu autoconf automake texinfo bison send-pr gprof rcs guile perl itcl tix gnuserv"
+    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
 
@@ -396,7 +396,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"
@@ -488,11 +488,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 tix readline mmalloc libgui itcl gdb ld"
+    noconfigdirs="$noconfigdirs readline mmalloc libgui itcl gdb ld"
     ;;
   i[[3456]]86-*-coff | i[[3456]]86-*-elf)
     noconfigdirs="$noconfigdirs ${libgcj}"
@@ -526,7 +526,7 @@
     # Can't build gdb for mingw32 if not native.
     case "${host}" in
       i[[3456]]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
     ;;    
@@ -539,7 +539,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
     ;;    
@@ -592,13 +592,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}"
@@ -1677,7 +1677,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="" ;;

Re: [toplevel patch] delete references to tix

[Keith Seitz]

On Mon, 2003-03-10 at 10:22, Nathanael Nerode wrote:
> Insight ripped tix out of CVS.  Time to get rid of it I think.  OK?
> 
> 	* Makefile.def: Remove references to tix.
> 	* Makefile.tpl: Remove references to tix.
> 	* Makefile.in: Regenerate.
> 	* configure.in: Remove references to tix.
> 	* configure: Regenerate.

Be my guest!

Keith

Re: [toplevel patch] delete references to tix

[Alexandre Oliva]

On Mar 10, 2003, Nathanael Nerode <neroden@twcny.rr.com> wrote:

> Insight ripped tix out of CVS.  Time to get rid of it I think.

Maybe it's a bit too early?  Consider that someone might create a
unified tree out of the latest stable releases of the tools, as
described in the CrossGCC FAQ IIRC, and then, if it turns out that the
insight release still uses tix, but the toplevel no longer does, we
lose.

I fail to see the point of removing stuff from the top-level just
because it appears to be unused.  The top-level tree is not just for
the tools, it was originally meant as a way to enable one to build a
number of projects in a single step.  Now it fails badly at that,
thanks to your `clean-ups' to ease the autoconfiscation.  Removing a
number of those projects has given us no benefit whatsoever, and
actually made the top-level less useful than it was.  I'm sure it
wouldn't have been that hard to keep a few additional host_modules,
even if they didn't appear to be used any more.

I wish you were *far* more conservative when considering whether to
remove something from the tree.  It's not like it's hard to put stuff
back in, but then we'll be going in circles.

Please defer the installation of this patch until a stable release of
gdb/insight is out there that no longer uses tix.  Ok to install 1-2
weeks afterwards.

-- 
Alexandre Oliva   Enjoy Guarana', see http://www.ic.unicamp.br/~oliva/
Red Hat GCC Developer                 aoliva@{redhat.com, gcc.gnu.org}
CS PhD student at IC-Unicamp        oliva@{lsd.ic.unicamp.br, gnu.org}
Free Software Evangelist                Professional serial bug killer

Re: [toplevel patch] delete references to tix

[Martin M. Hunt]

On Mon, 2003-03-10 at 11:06, Alexandre Oliva wrote:

> Please defer the installation of this patch until a stable release of
> gdb/insight is out there that no longer uses tix.  Ok to install 1-2
> weeks afterwards.

Insight hasn't used tix for 4-5 months.

Martin

Re: [toplevel patch] delete references to tix

[Alexandre Oliva]

On Mar 10, 2003, "Martin M. Hunt" <hunt@redhat.com> wrote:

> On Mon, 2003-03-10 at 11:06, Alexandre Oliva wrote:
>> Please defer the installation of this patch until a stable release of
>> gdb/insight is out there that no longer uses tix.  Ok to install 1-2
>> weeks afterwards.

> Insight hasn't used tix for 4-5 months.

I don't think there's a public release that doens't include tix yet.
Am I mistaken?  If so, I stand corrected, and the patch may very well
go in now.

-- 
Alexandre Oliva   Enjoy Guarana', see http://www.ic.unicamp.br/~oliva/
Red Hat GCC Developer                 aoliva@{redhat.com, gcc.gnu.org}
CS PhD student at IC-Unicamp        oliva@{lsd.ic.unicamp.br, gnu.org}
Free Software Evangelist                Professional serial bug killer

Re: [toplevel patch] delete references to tix

[Martin M. Hunt]

On Mon, 2003-03-10 at 11:18, Alexandre Oliva wrote:

> I don't think there's a public release that doens't include tix yet.
> Am I mistaken?  If so, I stand corrected, and the patch may very well
> go in now.

The last official release, gdb 5.3 had an Insight that still used tix.  
I don't recommend this version to Insight users, but I see your point
about builds.  I guess someone might still need the tix toplevel stuff;
I just left it in out of laziness and because it didn't affect anything.

Martin

Re: [toplevel patch] delete references to tix

[Andrew Cagney]

> On Mon, 2003-03-10 at 11:18, Alexandre Oliva wrote:
> 
> 
>> I don't think there's a public release that doens't include tix yet.
>> Am I mistaken?  If so, I stand corrected, and the patch may very well
>> go in now.
> 
> 
> The last official release, gdb 5.3 had an Insight that still used tix.  
> I don't recommend this version to Insight users, but I see your point
> about builds.  I guess someone might still need the tix toplevel stuff;
> I just left it in out of laziness and because it didn't affect anything.

Alex wrote:

 > Maybe it's a bit too early?  Consider that someone might create a
> unified tree out of the latest stable releases of the tools, as
> described in the CrossGCC FAQ IIRC, and then, if it turns out that the
> insight release still uses tix, but the toplevel no longer does, we
> lose.

I think `they loose'.  The senario you describe is, at this stage, 
strictly hypothetical.  It only occures if, at some future stage, a 
binutils or gcc containing this change is made before GDB.

Besides, such edge conditions should be covered by the CrossGCC FAQ (has 
that really not been changed in >3 years?).

enjoy,
Andrew

Re: [toplevel patch] delete references to tix

[Alexandre Oliva]

On Mar 10, 2003, Andrew Cagney <ac131313@redhat.com> wrote:

> It only occures if, at some future stage, a binutils or gcc
> containing this change is made before GDB.

Right.

> Besides, such edge conditions should be covered by the CrossGCC FAQ
> (has that really not been changed in >3 years?).

It moved to a Wiki base, IIRC.

Anyway, I don't see the point of removing support for building
packages from the tree.  I can still legitimately want to build Tix
along with Tcl, Tk and Itcl.  IMHO, the fact that the tools in src no
longer use Tix doesn't imply, to me, that the infrastructure to build
Tix should no longer be there.  It costs very little (or nothing) to
keep it working.  Removing it just because we can doesn't sound like
such a good idea to me.  Maybe I'm being overly conservative?  If more
people think so, please let me know, I'm always open to change.

-- 
Alexandre Oliva   Enjoy Guarana', see http://www.ic.unicamp.br/~oliva/
Red Hat GCC Developer                 aoliva@{redhat.com, gcc.gnu.org}
CS PhD student at IC-Unicamp        oliva@{lsd.ic.unicamp.br, gnu.org}
Free Software Evangelist                Professional serial bug killer

Re: [toplevel patch] delete references to tix

[Martin M. Hunt]

On Mon, 2003-03-10 at 16:20, Alexandre Oliva wrote:
> On Mar 10, 2003, Andrew Cagney <ac131313@redhat.com> wrote:
> 
> > It only occures if, at some future stage, a binutils or gcc
> > containing this change is made before GDB.
> 
> Right.
> 
> > Besides, such edge conditions should be covered by the CrossGCC FAQ
> > (has that really not been changed in >3 years?).
> 
> It moved to a Wiki base, IIRC.
> 
> Anyway, I don't see the point of removing support for building
> packages from the tree.  I can still legitimately want to build Tix
> along with Tcl, Tk and Itcl.  IMHO, the fact that the tools in src no
> longer use Tix doesn't imply, to me, that the infrastructure to build
> Tix should no longer be there.  It costs very little (or nothing) to
> keep it working. 

Just so we're clear on this, you can only build tix with old releases of
the tools, along with old tcl/tk, etc.  Tix hasn't been buildable out of
CVS for months and has been removed for that reason and because it was
no longer used.

Martin

Re: [toplevel patch] delete references to tix

[David Carlton]

On 10 Mar 2003 21:20:37 -0300, Alexandre Oliva <aoliva@redhat.com> said:

> Anyway, I don't see the point of removing support for building
> packages from the tree.  I can still legitimately want to build Tix
> along with Tcl, Tk and Itcl.  IMHO, the fact that the tools in src no
> longer use Tix doesn't imply, to me, that the infrastructure to build
> Tix should no longer be there.  It costs very little (or nothing) to
> keep it working.  Removing it just because we can doesn't sound like
> such a good idea to me.  Maybe I'm being overly conservative?  If more
> people think so, please let me know, I'm always open to change.

I've forgotten (or not entirely understood) what you're advocating,
but if you're talking about keeping around directories as opposed to
just config stuff, then it imposes a cost when they get downloaded
unnecessarily, when they get compiled unnecessarily.  So I don't want
checking out insight+dejagnu to download random packages just because
they once were necessary.  (Of course, the tix directory is now
deleted, so that's not applicable here.)

David Carlton
carlton@math.stanford.edu

Re: [toplevel patch] delete references to tix

[Alexandre Oliva]

On Mar 10, 2003, David Carlton <carlton@math.stanford.edu> wrote:

> I've forgotten (or not entirely understood) what you're advocating,

Just for leaving top-level config stuff alone that may help building
packages, even though they're not in either GCC or src CVS repo.
Nothing to do with keeping the actual packages around.

-- 
Alexandre Oliva   Enjoy Guarana', see http://www.ic.unicamp.br/~oliva/
Red Hat GCC Developer                 aoliva@{redhat.com, gcc.gnu.org}
CS PhD student at IC-Unicamp        oliva@{lsd.ic.unicamp.br, gnu.org}
Free Software Evangelist                Professional serial bug killer

AW: [toplevel patch] delete references to tix

[Carsten Schlote]

Hi,

> unnecessarily, when they get compiled unnecessarily.  So I don't want
> checking out insight+dejagnu to download random packages just because
> they once were necessary.  

I fully agree. I downloaded the complete CVS some days ago, just to find out, that TIX neither builds with CygWin nor was needed anyway (don't worry, I also use linux, bsd etc..)

I'm really only interested in the latest version of GDB and insight. All obsolete modules should be removed after some suitable time (max. 6 months). If people should need TIX for some reason I guess there are other repositories, where you can still get it.

I'm a happy ADSL user. But with my old modem and no flat rate, I'd really hate obsolete crap to be downloaded...

Carsten