* [PATCH] Don't try building gdb against guile-2.2
@ 2017-10-08 16:47 Simon Marchi
2017-10-08 17:14 ` Eli Zaretskii
0 siblings, 1 reply; 6+ messages in thread
From: Simon Marchi @ 2017-10-08 16:47 UTC (permalink / raw)
To: gdb-patches; +Cc: Simon Marchi
GDB currently doesn't build with Guile 2.2 (see PR 21104). If one has
both Guile 2.2 and 2.0 installed, GDB will pick up Guile 2.2 first and
fail building. Until somebody does the work of adapting the GDB code to
Guile 2.2, we should not try using it. This patch therefore removes it
from configure.
gdb/ChangeLog:
* configure.ac (try_guile_versions): Remove guile-2.2.
* configure: Regenerate.
---
gdb/configure | 4 ++--
gdb/configure.ac | 2 +-
2 files changed, 3 insertions(+), 3 deletions(-)
diff --git a/gdb/configure b/gdb/configure
index 4d473cc934..6b445e4f67 100755
--- a/gdb/configure
+++ b/gdb/configure
@@ -5950,7 +5950,7 @@ DEPDIR="${am__leading_dot}deps"
ac_config_commands="$ac_config_commands depdir"
-# Create sub-directories for objects and depedencies.
+# Create sub-directories for objects and dependencies.
CONFIG_SRC_SUBDIR="arch"
@@ -10313,7 +10313,7 @@ fi
-try_guile_versions="guile-2.2 guile-2.0"
+try_guile_versions="guile-2.0"
have_libguile=no
case "${with_guile}" in
no)
diff --git a/gdb/configure.ac b/gdb/configure.ac
index 4d8d6c7e76..4e64c80f62 100644
--- a/gdb/configure.ac
+++ b/gdb/configure.ac
@@ -1138,7 +1138,7 @@ AC_MSG_RESULT([$with_guile])
dnl We check guile with pkg-config.
AC_PATH_PROG(pkg_config_prog_path, pkg-config, missing)
-try_guile_versions="guile-2.2 guile-2.0"
+try_guile_versions="guile-2.0"
have_libguile=no
case "${with_guile}" in
no)
--
2.14.2
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH] Don't try building gdb against guile-2.2
2017-10-08 16:47 [PATCH] Don't try building gdb against guile-2.2 Simon Marchi
@ 2017-10-08 17:14 ` Eli Zaretskii
2017-10-08 17:20 ` Simon Marchi
0 siblings, 1 reply; 6+ messages in thread
From: Eli Zaretskii @ 2017-10-08 17:14 UTC (permalink / raw)
To: Simon Marchi; +Cc: gdb-patches
> From: Simon Marchi <simon.marchi@polymtl.ca>
> Cc: Simon Marchi <simon.marchi@polymtl.ca>
> Date: Sun, 8 Oct 2017 12:47:03 -0400
>
> GDB currently doesn't build with Guile 2.2 (see PR 21104). If one has
> both Guile 2.2 and 2.0 installed, GDB will pick up Guile 2.2 first and
> fail building. Until somebody does the work of adapting the GDB code to
> Guile 2.2, we should not try using it. This patch therefore removes it
> from configure.
>
> gdb/ChangeLog:
>
> * configure.ac (try_guile_versions): Remove guile-2.2.
> * configure: Regenerate.
Should we perhaps have a configure-time option, by default off, to use
Guile 2.2? Otherwise, how would that "somebody" work on fixing this
problem, if they cannot even try building with Guile 2.2?
Thanks.
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH] Don't try building gdb against guile-2.2
2017-10-08 17:14 ` Eli Zaretskii
@ 2017-10-08 17:20 ` Simon Marchi
2017-10-08 17:42 ` Eli Zaretskii
0 siblings, 1 reply; 6+ messages in thread
From: Simon Marchi @ 2017-10-08 17:20 UTC (permalink / raw)
To: Eli Zaretskii; +Cc: gdb-patches
On 2017-10-08 13:13, Eli Zaretskii wrote:
>> From: Simon Marchi <simon.marchi@polymtl.ca>
>> Cc: Simon Marchi <simon.marchi@polymtl.ca>
>> Date: Sun, 8 Oct 2017 12:47:03 -0400
>>
>> GDB currently doesn't build with Guile 2.2 (see PR 21104). If one has
>> both Guile 2.2 and 2.0 installed, GDB will pick up Guile 2.2 first and
>> fail building. Until somebody does the work of adapting the GDB code
>> to
>> Guile 2.2, we should not try using it. This patch therefore removes
>> it
>> from configure.
>>
>> gdb/ChangeLog:
>>
>> * configure.ac (try_guile_versions): Remove guile-2.2.
>> * configure: Regenerate.
>
> Should we perhaps have a configure-time option, by default off, to use
> Guile 2.2? Otherwise, how would that "somebody" work on fixing this
> problem, if they cannot even try building with Guile 2.2?
>
> Thanks.
They can pass --with-guile=guile-2.2. try_guile_versions, AFAIU,
contains the version to try by default if the version to use is not
explicitly passed (--with-guile or --with-guile=yes).
Simon
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH] Don't try building gdb against guile-2.2
2017-10-08 17:20 ` Simon Marchi
@ 2017-10-08 17:42 ` Eli Zaretskii
0 siblings, 0 replies; 6+ messages in thread
From: Eli Zaretskii @ 2017-10-08 17:42 UTC (permalink / raw)
To: Simon Marchi; +Cc: gdb-patches
> Date: Sun, 08 Oct 2017 13:20:28 -0400
> From: Simon Marchi <simon.marchi@polymtl.ca>
> Cc: gdb-patches@sourceware.org
>
> > Should we perhaps have a configure-time option, by default off, to use
> > Guile 2.2? Otherwise, how would that "somebody" work on fixing this
> > problem, if they cannot even try building with Guile 2.2?
> >
> > Thanks.
>
> They can pass --with-guile=guile-2.2. try_guile_versions, AFAIU,
> contains the version to try by default if the version to use is not
> explicitly passed (--with-guile or --with-guile=yes).
Ah, okay. That's good enough, I think.
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH] Don't try building gdb against guile-2.2
2017-10-09 16:42 Doug Evans via gdb-patches
@ 2017-10-09 16:52 ` Simon Marchi
0 siblings, 0 replies; 6+ messages in thread
From: Simon Marchi @ 2017-10-09 16:52 UTC (permalink / raw)
To: Doug Evans, Eli Zaretskii; +Cc: gdb-patches
On 2017-10-09 12:42 PM, Doug Evans wrote:
> Eli Zaretskii writes:
> > > Date: Sun, 08 Oct 2017 13:20:28 -0400
> > > From: Simon Marchi <simon.marchi@polymtl.ca>
> > > Cc: gdb-patches@sourceware.org
> > >
> > > > Should we perhaps have a configure-time option, by default off, to use
> > > > Guile 2.2? Otherwise, how would that "somebody" work on fixing this
> > > > problem, if they cannot even try building with Guile 2.2?
> > > >
> > > > Thanks.
> > >
> > > They can pass --with-guile=guile-2.2. try_guile_versions, AFAIU,
> > > contains the version to try by default if the version to use is not
> > > explicitly passed (--with-guile or --with-guile=yes).
> >
> > Ah, okay. That's good enough, I think.
>
> SGTM
>
Thanks, pushed.
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH] Don't try building gdb against guile-2.2
@ 2017-10-09 16:42 Doug Evans via gdb-patches
2017-10-09 16:52 ` Simon Marchi
0 siblings, 1 reply; 6+ messages in thread
From: Doug Evans via gdb-patches @ 2017-10-09 16:42 UTC (permalink / raw)
To: Eli Zaretskii; +Cc: Simon Marchi, gdb-patches
Eli Zaretskii writes:
> > Date: Sun, 08 Oct 2017 13:20:28 -0400
> > From: Simon Marchi <simon.marchi@polymtl.ca>
> > Cc: gdb-patches@sourceware.org
> >
> > > Should we perhaps have a configure-time option, by default off, to use
> > > Guile 2.2? Otherwise, how would that "somebody" work on fixing this
> > > problem, if they cannot even try building with Guile 2.2?
> > >
> > > Thanks.
> >
> > They can pass --with-guile=guile-2.2. try_guile_versions, AFAIU,
> > contains the version to try by default if the version to use is not
> > explicitly passed (--with-guile or --with-guile=yes).
>
> Ah, okay. That's good enough, I think.
SGTM
^ permalink raw reply [flat|nested] 6+ messages in thread
end of thread, other threads:[~2017-10-09 16:52 UTC | newest]
Thread overview: 6+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2017-10-08 16:47 [PATCH] Don't try building gdb against guile-2.2 Simon Marchi
2017-10-08 17:14 ` Eli Zaretskii
2017-10-08 17:20 ` Simon Marchi
2017-10-08 17:42 ` Eli Zaretskii
2017-10-09 16:42 Doug Evans via gdb-patches
2017-10-09 16:52 ` Simon Marchi
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox