[PATCH 2 of 2] debug/cross-gdb: check host dependencies

Benoît THÉBAUDEAU benoit.thebaudeau@advansee.com
Wed Jun 8 14:08:00 GMT 2011

# HG changeset patch
# User "Benoît THÉBAUDEAU" <benoit.thebaudeau@advansee.com>
# Date 1307540863 -7200
# Node ID 9ff0761e151e9aa6f2e3e81d2add55f641893edf
# Parent  a5856225d851cae256be03278343c562b8ebdc54
debug/cross-gdb: check host dependencies

Cross-gdb depends on expat and python. If either is missing, cross-gdb will
build successfully, but lacking some features.

Especially, if expat is missing, cross-gdb will be unable to parse the target
description, which may lead to runtime malfunctions and the following GDB
"Can not parse XML target description; XML support was disabled at compile time"

Hence, expat should be considered mandatory.

On the other hand, the features missing without python are not critical, so
python should not be considered mandatory.

This patch does the following:
 - At configure time, warn the user if either expat or python is missing.
 - In menuconfig, disable the static build options regarding cross-gdb if no
   static version of expat is available, and disable cross-gdb if expat is

Signed-off-by: "Benoît THÉBAUDEAU" <benoit.thebaudeau@advansee.com>

diff --git a/config/debug/gdb.in.cross b/config/debug/gdb.in.cross
--- a/config/debug/gdb.in.cross
+++ b/config/debug/gdb.in.cross
@@ -3,10 +3,15 @@
+comment "Cross-gdb - disabled (requires expat; re-run ./configure to enable)"
+    depends on !CONFIGURE_has_expat
 config GDB_CROSS
     prompt "Cross-gdb"
     default y
+    depends on CONFIGURE_has_expat
+    depends on !STATIC_TOOLCHAIN || CONFIGURE_has_static_expat
       Build and install a cross-gdb for the target, to run on host.
@@ -16,6 +21,7 @@
     prompt "Build a static cross gdb"
+    depends on CONFIGURE_has_static_expat
       A static cross gdb can be usefull if you debug on a machine that is
       not the one that is used to compile the toolchain.
diff --git a/configure b/configure
--- a/configure
+++ b/configure
@@ -462,6 +462,34 @@
              err="static 'libstdc++' is needed to statically link the toolchain's executables" \
+expat_libs="$( for x in so dylib a; do \
+                   printf "libexpat.$x "; \
+               done \
+             )"
+has_or_warn  inc="expat.h" \
+             lib="${expat_libs}" \
+             err="The 'expat' header file and library are needed to link cross-gdb's executables" \
+             kconfig=has_expat
+# Yes, we may be checking twice for libexpat.a
+# The first is because we need one instance of libexpat (shared or static)
+# because it is needed for cross-gdb; the second is because the static version
+# is required for static-linking, and if missing, the option is removed.
+has_or_warn  lib="libexpat.a" \
+             err="static 'expat' is needed to statically link cross-gdb's executables" \
+             kconfig=has_static_expat
+for v in 7 6 5 4; do
+    python_incs="${python_incs}$( printf "python2.$v/Python.h " )"
+    python_libs="${python_libs}$( for x in so dylib a; do \
+                                      printf "libpython2.$v.$x "; \
+                                  done \
+                                )"
+has_or_warn  inc="${python_incs}" \
+             lib="${python_libs}" \
+             err="The 'python' header file and library are needed for some features of cross-gdb"
 # Compute the version string
diff --git a/scripts/build/debug/300-gdb.sh b/scripts/build/debug/300-gdb.sh
--- a/scripts/build/debug/300-gdb.sh
+++ b/scripts/build/debug/300-gdb.sh
@@ -139,6 +139,7 @@
             --prefix="${CT_PREFIX_DIR}"                 \
             --with-build-sysroot="${CT_SYSROOT_DIR}"    \
             --with-sysroot="${CT_SYSROOT_DIR}"          \
+            --with-expat=yes                            \
             --disable-werror                            \

More information about the crossgcc mailing list