This is the mail archive of the
insight@sources.redhat.com
mailing list for the Insight project.
Re: TODO list (REPOST)
- To: insight at sources dot redhat dot com
- Subject: Re: TODO list (REPOST)
- From: Mo DeJong <mdejong at cygnus dot com>
- Date: Tue, 10 Oct 2000 23:15:40 -0700 (PDT)
On Tue, 10 Oct 2000, Fernando Nasser wrote:
(Bunch of stuff about what to fix in insight)
With all this talk of "fixing" insight, I thought I
would bring up the old name issue again. The #1
question people have when starting out with
insight is "how do I run it?". People automatically
assume that the name of the executable is insight.
Lets just change the name of the executable to
insight, it is the right thing to do. This
also gets around the problem of typing % gdb
and getting insight when you expected to get
the cmd line gdb from the distro.
Here is my patch to implement the change
(minus the patch for configure itself).
Ok to check in?
Mo DeJong
Red Hat Inc
Index: ChangeLog
===================================================================
RCS file: /cvs/src/src/gdb/ChangeLog,v
retrieving revision 1.702
diff -u -r1.702 ChangeLog
--- ChangeLog 2000/10/10 05:17:25 1.702
+++ ChangeLog 2000/10/11 06:12:33
@@ -1,3 +1,11 @@
+2000-10-10 Mo DeJong <mdejong@redhat.com>
+
+ * Makefile.in: Add GDB variable, it will be set to
+ gdb or insight, depending on what mode is used.
+ * configure: Regen.
+ * configure.in: Add GDB variable, pass it to Makefile.in.
+ * gdbtk/README.GDBTK: Fixed minor little typo.
+
2000-10-09 Kevin Buettner <kevinb@redhat.com>
* remote-nindy.c (non_dle, nindy_xfer_inferior_memory): Protoize.
Index: Makefile.in
===================================================================
RCS file: /cvs/src/src/gdb/Makefile.in,v
retrieving revision 1.44
diff -u -r1.44 Makefile.in
--- Makefile.in 2000/10/08 22:31:22 1.44
+++ Makefile.in 2000/10/11 06:12:35
@@ -62,6 +62,9 @@
WINDRES = @WINDRES@
MIG = @MIG@
+# Name of gdb executable, set to "insight" in gdbtk mode
+GDB = @GDB@
+
# Flags that describe where you can find the termcap library.
# This can be overridden in the host Makefile fragment file.
TERMCAP = @TERM_LIB@
@@ -669,7 +672,7 @@
.c.o:
$(CC) -c $(INTERNAL_CFLAGS) $<
-all: gdb$(EXEEXT) $(CONFIG_ALL)
+all: $(GDB)$(EXEEXT) $(CONFIG_ALL)
@$(MAKE) $(FLAGS_TO_PASS) DO=all "DODIRS=`echo $(SUBDIRS) | sed
's/testsuite//'`" subdir_do
installcheck:
@@ -700,23 +703,23 @@
install: all install-only
install-only: $(CONFIG_INSTALL)
transformed_name=`t='$(program_transform_name)'; \
- echo gdb | sed -e $$t` ; \
+ echo $(GDB) | sed -e $$t` ; \
if test "x$$transformed_name" = x; then \
- transformed_name=gdb ; \
+ transformed_name=$(GDB) ; \
else \
true ; \
fi ; \
$(srcdir)/../mkinstalldirs $(bindir) ; \
- $(INSTALL_PROGRAM) gdb$(EXEEXT)
$(bindir)/$$transformed_name$(EXEEXT) ; \
+ $(INSTALL_PROGRAM) $(GDB)$(EXEEXT)
$(bindir)/$$transformed_name$(EXEEXT) ; \
$(srcdir)/../mkinstalldirs $(man1dir) ; \
$(INSTALL_DATA) $(srcdir)/gdb.1 $(man1dir)/$$transformed_name.1
@$(MAKE) DO=install "DODIRS=$(SUBDIRS)" $(FLAGS_TO_PASS)
subdir_do
uninstall: force $(CONFIG_UNINSTALL)
transformed_name=`t='$(program_transform_name)'; \
- echo gdb | sed -e $$t` ; \
+ echo $(GDB) | sed -e $$t` ; \
if test "x$$transformed_name" = x; then \
- transformed_name=gdb ; \
+ transformed_name=$(GDB) ; \
else \
true ; \
fi ; \
@@ -783,9 +786,9 @@
init.o: init.c $(defs_h) $(call_cmds_h)
# Removing the old gdb first works better if it is running, at least on
SunOS.
-gdb$(EXEEXT): main.o libgdb.a $(ADD_DEPS) $(CDEPS) $(TDEPLIBS)
- rm -f gdb$(EXEEXT)
- $(HLDENV) $(CC_LD) $(INTERNAL_LDFLAGS) $(WIN32LDAPP) -o
gdb$(EXEEXT) main.o libgdb.a $(TDEPLIBS) $(TUI_LIBRARY) $(CLIBS) $(LOADLIBES)
+$(GDB)$(EXEEXT): main.o libgdb.a $(ADD_DEPS) $(CDEPS) $(TDEPLIBS)
+ rm -f $(GDB)$(EXEEXT)
+ $(HLDENV) $(CC_LD) $(INTERNAL_LDFLAGS) $(WIN32LDAPP) -o
$(GDB)$(EXEEXT) main.o libgdb.a $(TDEPLIBS) $(TUI_LIBRARY) $(CLIBS)
$(LOADLIBES)
nlm: force
rootme=`pwd`; export rootme; $(MAKE) $(TARGET_FLAGS_TO_PASS)
DO=all DODIRS=nlm subdir_do
@@ -838,7 +841,7 @@
# in another process.
gdb1$(EXEEXT): gdb$(EXEEXT)
rm -f gdb1$(EXEEXT)
- cp gdb$(EXEEXT) gdb1$(EXEEXT)
+ cp $(GDB)$(EXEEXT) gdb1$(EXEEXT)
# FIXME. These are not generated by "make depend" because they only are
there
# for some machines.
@@ -883,7 +886,7 @@
@$(MAKE) $(FLAGS_TO_PASS) DO=clean "DODIRS=$(SUBDIRS)" subdir_do
rm -f *.o *.a $(ADD_FILES) *~ init.c-tmp init.l-tmp
rm -f init.c version.c
- rm -f gdb$(EXEEXT) core make.log
+ rm -f $(GDB)$(EXEEXT) core make.log
rm -f gdb[0-9]$(EXEEXT)
# This used to depend on c-exp.tab.c m2-exp.tab.c TAGS
Index: configure.in
===================================================================
RCS file: /cvs/src/src/gdb/configure.in,v
retrieving revision 1.44
diff -u -r1.44 configure.in
--- configure.in 2000/09/11 17:57:07 1.44
+++ configure.in 2000/10/11 06:12:35
@@ -759,9 +759,13 @@
configdir="unix"
+# This will be set to "insight" if using gdbtk
+GDB=gdb
+
GDBTKLIBS=
if test "${enable_gdbtk}" = "yes"; then
+ GDB=insight
CY_AC_PATH_TCLCONFIG
if test -z "${no_tcl}"; then
CY_AC_LOAD_TCLCONFIG
@@ -869,6 +873,7 @@
AC_SUBST(ITK_DEPS)
AC_SUBST(TIXLIB)
AC_SUBST(TIX_DEPS)
+AC_SUBST(GDB)
AC_SUBST(GDBTKLIBS)
AC_SUBST(GDBTK_CFLAGS)
Index: gdbtk/README.GDBTK
===================================================================
RCS file: /cvs/src/src/gdb/gdbtk/README.GDBTK,v
retrieving revision 1.2
diff -u -r1.2 README.GDBTK
--- README.GDBTK 2000/04/17 20:20:20 1.2
+++ README.GDBTK 2000/10/11 06:12:36
@@ -10,7 +10,7 @@
============
GDBtk is a version of GDB that uses Tcl/Tk to implement a graphical
-user inter-face. It is a fully integrated GUI, not a separate
+user interface. It is a fully integrated GUI, not a separate
front-end program. The interface consists of several seperate
windows, which use standard elements like buttons, scrollbars, entry
boxes and such to create a fairly easy to use interface. Each window