* [rfa/toplevel] Conditionalize GDB_TK
@ 2003-01-03 19:23 Andrew Cagney
2003-01-03 19:46 ` DJ Delorie
2003-01-04 0:45 ` Michael Snyder
0 siblings, 2 replies; 4+ messages in thread
From: Andrew Cagney @ 2003-01-03 19:23 UTC (permalink / raw)
To: gcc-patches; +Cc: insight, gdb-patches
[-- Attachment #1: Type: text/plain, Size: 322 bytes --]
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).
ok?
Andrew
PS: Disclaimer, I cut/paste the configure changes. Perhaphs someone
able to regenerate configure should commit this?
[-- Attachment #2: diffs --]
[-- Type: text/plain, Size: 2369 bytes --]
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.
^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: [rfa/toplevel] Conditionalize GDB_TK
2003-01-03 19:23 [rfa/toplevel] Conditionalize GDB_TK Andrew Cagney
@ 2003-01-03 19:46 ` DJ Delorie
2003-01-06 18:38 ` Andrew Cagney
2003-01-04 0:45 ` Michael Snyder
1 sibling, 1 reply; 4+ messages in thread
From: DJ Delorie @ 2003-01-03 19:46 UTC (permalink / raw)
To: ac131313; +Cc: gcc-patches, insight, gdb-patches
> The attached 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).
>
> ok?
Ok.
> PS: Disclaimer, I cut/paste the configure changes. Perhaps someone
> able to regenerate configure should commit this?
You don't have autoconf 2.13?
^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: [rfa/toplevel] Conditionalize GDB_TK
2003-01-03 19:23 [rfa/toplevel] Conditionalize GDB_TK Andrew Cagney
2003-01-03 19:46 ` DJ Delorie
@ 2003-01-04 0:45 ` Michael Snyder
1 sibling, 0 replies; 4+ messages in thread
From: Michael Snyder @ 2003-01-04 0:45 UTC (permalink / raw)
To: Andrew Cagney; +Cc: gcc-patches, insight, gdb-patches
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.
^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: [rfa/toplevel] Conditionalize GDB_TK
2003-01-03 19:46 ` DJ Delorie
@ 2003-01-06 18:38 ` Andrew Cagney
0 siblings, 0 replies; 4+ messages in thread
From: Andrew Cagney @ 2003-01-06 18:38 UTC (permalink / raw)
To: DJ Delorie; +Cc: gcc-patches, insight, gdb-patches
>> The attached 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).
>>
>> ok?
>
>
> Ok.
Thanks.
>> PS: Disclaimer, I cut/paste the configure changes. Perhaps someone
>> able to regenerate configure should commit this?
>
>
> You don't have autoconf 2.13?
Which 2.13? :-) Anyway I found the correct version, re-generated the
file and committed (both GCC and SRC).
Andrew
^ permalink raw reply [flat|nested] 4+ messages in thread
end of thread, other threads:[~2003-01-06 18:38 UTC | newest]
Thread overview: 4+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2003-01-03 19:23 [rfa/toplevel] Conditionalize GDB_TK Andrew Cagney
2003-01-03 19:46 ` DJ Delorie
2003-01-06 18:38 ` Andrew Cagney
2003-01-04 0:45 ` Michael Snyder
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).