Message ID | alpine.LFD.2.21.2002081532520.18621@redsun52.ssa.fujisawa.hgst.com |
---|---|
State | Superseded |
Headers | show |
>>>>> "Maciej" == Maciej W Rozycki <macro@wdc.com> writes:
Maciej> Correct fallout from commit 919adfe84092 ("Move gdbserver to top level")
Maciej> and revert to not building `gdbserver' in a cross-configuration, that is
Maciej> where host != target, matching the documented behaviour. We have no way
Maciej> to support non-native `gdbserver', and native `gdbserver' is usually of
Maciej> no use with cross-GDB of the chosen host.
Pedro had a different way to do this, that keeps the decision under
gdbserver's control:
https://sourceware.org/ml/gdb-patches/2020-02/msg00383.html
Also note that I haven't sent the top-level configury patches upstream
to gcc yet. I am going to do this all at once, at the end. I think
there's one more patch that will be needed, once gdbserver depends on
the top-level gdbsupport.
Tom
Index: binutils-gdb/configure =================================================================== --- binutils-gdb.orig/configure +++ binutils-gdb/configure @@ -3538,12 +3538,13 @@ case "${target}" in ;; esac -# Only allow gdbserver on some systems. +# Only allow native gdbserver and then only on some systems. if test -d ${srcdir}/gdbserver; then if test x$enable_gdbserver = x; then { $as_echo "$as_me:${as_lineno-$LINENO}: checking for gdbserver support" >&5 $as_echo_n "checking for gdbserver support... " >&6; } - if (srcdir=${srcdir}/gdbserver; \ + if test x${is_cross_compiler} = xyes \ + || (srcdir=${srcdir}/gdbserver; \ . ${srcdir}/configure.srv; \ test -n "$UNSUPPORTED") then Index: binutils-gdb/configure.ac =================================================================== --- binutils-gdb.orig/configure.ac +++ binutils-gdb/configure.ac @@ -782,11 +782,12 @@ case "${target}" in ;; esac -# Only allow gdbserver on some systems. +# Only allow native gdbserver and then only on some systems. if test -d ${srcdir}/gdbserver; then if test x$enable_gdbserver = x; then AC_MSG_CHECKING([for gdbserver support]) - if (srcdir=${srcdir}/gdbserver; \ + if test x${is_cross_compiler} = xyes \ + || (srcdir=${srcdir}/gdbserver; \ . ${srcdir}/configure.srv; \ test -n "$UNSUPPORTED") then