Don't try building gdb against guile-2.2
Commit Message
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(-)
Comments
> 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.
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
> 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.
@@ -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)
@@ -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)