From: Mo DeJong <mdejong@cygnus.com>
To: insight@sources.redhat.com
Subject: Re: TODO list (REPOST)
Date: Tue, 10 Oct 2000 23:15:00 -0000 [thread overview]
Message-ID: <Pine.SOL.3.91.1001010143700.29621I-100000@cse.cygnus.com> (raw)
In-Reply-To: <39E333E8.641C0E85@cygnus.com>
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
next prev parent reply other threads:[~2000-10-10 23:15 UTC|newest]
Thread overview: 13+ messages / expand[flat|nested] mbox.gz Atom feed top
2000-10-10 8:21 Fernando Nasser
2000-10-10 23:15 ` Mo DeJong [this message]
2000-10-11 6:25 ` Grant Edwards
2000-10-11 6:54 ` Fernando Nasser
2000-10-11 10:26 ` Tom Tromey
2000-10-11 10:38 ` Grant Edwards
2000-10-11 10:39 ` Mo DeJong
2000-10-11 14:17 ` Mo DeJong
2000-10-11 14:44 ` Fernando Nasser
2000-10-11 14:53 ` Mo DeJong
2000-10-11 15:03 ` Fernando Nasser
2000-10-11 15:31 ` Fix to Breakpoint right-click menu bug [Was: TODO list (REPOST)] Fernando Nasser
2000-10-11 14:56 ` RMS in the press " Fernando Nasser
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=Pine.SOL.3.91.1001010143700.29621I-100000@cse.cygnus.com \
--to=mdejong@cygnus.com \
--cc=insight@sources.redhat.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).