This is the mail archive of the
gdb-patches@sources.redhat.com
mailing list for the GDB project.
Re: [rfa/toplevel] Conditionalize GDB_TK
- From: Michael Snyder <msnyder at redhat dot com>
- To: Andrew Cagney <ac131313 at redhat dot com>
- Cc: gcc-patches at gcc dot gnu dot org, insight at sources dot redhat dot com, gdb-patches at sources dot redhat dot com
- Date: Fri, 03 Jan 2003 16:45:25 -0800
- Subject: Re: [rfa/toplevel] Conditionalize GDB_TK
- Organization: Red Hat, Inc.
- References: <3E15E32C.2050405@redhat.com>
Andrew Cagney wrote:
>
> Hello,
>
> The attatched only adds the tk directories to the gdb dependency list
> when the corresponding tk files are present. That way `all-gdb' doesn't
> build tk files (except when needed).
Thank you!
> ok?
> Andrew
>
> PS: Disclaimer, I cut/paste the configure changes. Perhaphs someone
> able to regenerate configure should commit this?
>
> -------------------------------------------------------------------------------
> 2003-01-03 Andrew Cagney <ac131313@redhat.com>
>
> * configure.in (GDB_TK): Add tcl directories conditional on
> gdb/gdbtk directory being present.
> * configure: Regenerate.
>
> Index: configure
> ===================================================================
> RCS file: /cvs/src/src/configure,v
> retrieving revision 1.40
> diff -u -r1.40 configure
> --- configure 31 Dec 2002 18:00:11 -0000 1.40
> +++ configure 3 Jan 2003 17:48:45 -0000
> @@ -2334,11 +2334,23 @@
> # so we should be able to put the 'maybe's in unconditionally and
> # 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"
> case "$enable_gdbtk" in
> no)
> GDB_TK="" ;;
> + yes)
> + GDB_TK="${gdb_tk}" ;;
> *)
> - GDB_TK="maybe-all-tcl maybe-all-tk maybe-all-itcl maybe-all-tix maybe-all-libgui" ;;
> + # Only add the dependency on gdbtk when GDBtk is part of the gdb
> + # distro. Eventually someone will fix this and move Insight, nee
> + # gdbtk to a separate directory.
> + if test -d ${srcdir}/gdb/gdbtk ; then
> + GDB_TK="${gdb_tk}"
> + else
> + GDB_TK=""
> + fi
> + ;;
> esac
>
> # Create the 'maybe dependencies'. This uses a temporary file.
> Index: configure.in
> ===================================================================
> RCS file: /cvs/src/src/configure.in,v
> retrieving revision 1.137
> diff -u -r1.137 configure.in
> --- configure.in 31 Dec 2002 18:00:11 -0000 1.137
> +++ configure.in 3 Jan 2003 17:48:45 -0000
> @@ -1677,11 +1677,23 @@
> # so we should be able to put the 'maybe's in unconditionally and
> # 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"
> case "$enable_gdbtk" in
> no)
> GDB_TK="" ;;
> + yes)
> + GDB_TK="${gdb_tk}" ;;
> *)
> - GDB_TK="maybe-all-tcl maybe-all-tk maybe-all-itcl maybe-all-tix maybe-all-libgui" ;;
> + # Only add the dependency on gdbtk when GDBtk is part of the gdb
> + # distro. Eventually someone will fix this and move Insight, nee
> + # gdbtk to a separate directory.
> + if test -d ${srcdir}/gdb/gdbtk ; then
> + GDB_TK="${gdb_tk}"
> + else
> + GDB_TK=""
> + fi
> + ;;
> esac
>
> # Create the 'maybe dependencies'. This uses a temporary file.