This is the mail archive of the
insight@sourceware.cygnus.com
mailing list for the Insight project.
Re: RFC: More fixes to insights ln -s
- To: Andrew Cagney <ac131313 at cygnus dot com>
- Subject: Re: RFC: More fixes to insights ln -s
- From: James Ingham <jingham at cygnus dot com>
- Date: Wed, 15 Mar 2000 10:24:36 -0800 (PST)
- Cc: "Insight (GDB GUI)" <insight at sourceware dot cygnus dot com>, GDB Patches <gdb-patches at sourceware dot cygnus dot com>
- References: <38CF37E7.AADF52CF@cygnus.com>
Andrew,
I haven't really been paying that much attention to this configure
stuff of late, sorry... But now that I think of it, I am pretty sure
all these link machinations are wholly unnecessary. I munged the gdb
startup code around so that gdb will find its library files from the
build directory without using the link at all. Moreover, when you
fail at making the link gdb will still work in GUI mode. So maybe we
can just bag all this?
However, it would be good to make sure that this is not some wierdness
in my environment that makes it work. What about others? If you
delete the gdbtcl link in your build directory, can you still run gdb
in gui mode from the gdb build directory? If this is true for others
as well, then we should just drop the extra complexity.
Jim
> Hello,
>
> The attatched patch makes the make target ``all-gdbtk'' which creates a
> link more robust. Fernando noted that it issued a warning when it
> didn't need to. It could also trip up if there was an invalid old link.
>
> Ok?
>
> AndrewWed Mar 15 17:32:48 2000 Andrew Cagney <cagney@b1.cygnus.com>
>
> * Makefile.in (all-gdbtk): Check for an existing link/directory.
> Re-format warning message.
>
> Index: Makefile.in
> ===================================================================
> RCS file: /cvs/src/src/gdb/Makefile.in,v
> retrieving revision 1.12
> diff -p -r1.12 Makefile.in
> *** Makefile.in 2000/03/04 07:11:38 1.12
> --- Makefile.in 2000/03/15 07:09:53
> *************** fork-child.o: fork-child.c gdb_wait.h $(
> *** 1260,1273 ****
> $(inferior_h) target.h terminal.h gdbthread.h gdb_string.h
>
> all-gdbtk:
> ! if test "$(LN_S)" = "ln -s" -a ! -d gdbtcl/images ; then \
> ! echo linking ${srcdir}/gdbtk/library to gdbtcl ; \
> ! $(LN_S) ${srcdir}/gdbtk/library gdbtcl ; \
> ! else \
> ! echo Warning: Unable to link ${srcdir}/gdbtk/library to gdbtcl. ; \
> ! echo " " You will need to do a ; \
> ! echo " " make install before you are able to run the GUI. ; \
> ! fi
>
> clean-gdbtk:
> rm -f gdbtcl
> --- 1260,1278 ----
> $(inferior_h) target.h terminal.h gdbthread.h gdb_string.h
>
> all-gdbtk:
> ! @if test ! -d gdbtcl/images ; then \
> ! if test "$(LN_S)" = "ln -s" ; then \
> ! echo linking ${srcdir}/gdbtk/library to gdbtcl ; \
> ! rm -f gdbtcl ; \
> ! test ! -r gdbtcl || exit 1 ; \
> ! $(LN_S) ${srcdir}/gdbtk/library gdbtcl ; \
> ! else \
> ! echo "Warning:" ; \
> ! echo "Unable to link ${srcdir}/gdbtk/library to gdbtcl." ; \
> ! echo "You will need to do a \`make install' before you are" ; \
> ! echo "able to run the GUI." ; \
> ! fi ; \
> ! else true ; fi
>
> clean-gdbtk:
> rm -f gdbtcl