From eb687c65b33e35e282f152b1bf19163776e599dd Mon Sep 17 00:00:00 2001 From: "Yann E. MORIN\"" Date: Tue, 9 Dec 2008 22:02:20 +0000 Subject: Sanity-check CT_TARGET_VENDOR, CT_TARGET_ALIAS and CT_TARGET_ALIAS_SED_EXPR: - vendor and alias must not contain spaces - vendor must not contain dashes '-' - sed_expr must not generate an alias with a space in it /trunk/scripts/functions | 17 16 1 0 ++++++++++++++++- /trunk/config/toolchain.in | 1 1 0 0 + 2 files changed, 17 insertions(+), 1 deletion(-) diff --git a/config/toolchain.in b/config/toolchain.in index 89f3ffb..61d939d 100644 --- a/config/toolchain.in +++ b/config/toolchain.in @@ -33,6 +33,7 @@ config TARGET_VENDOR A tuple is of the form arch-vendor-kernel-system. You can set the second part, vendor, to whatever you see fit. Use a single word, or use underscores "_" to separate words. + Use neither dash nor space, as it breaks things. Keep the default (unkown) if you don't know better. diff --git a/scripts/functions b/scripts/functions index cffd6ba..6fbc9cf 100644 --- a/scripts/functions +++ b/scripts/functions @@ -697,7 +697,22 @@ CT_DoBuildTargetTuple() { CT_DoKernelTupleValues # Finish the target tuple construction - CT_TARGET=$(CT_DoConfigSub "${CT_TARGET_ARCH}-${CT_TARGET_VENDOR:-unknown}-${CT_TARGET_KERNEL}${CT_TARGET_KERNEL:+-}${CT_TARGET_SYS}") + CT_TARGET="${CT_TARGET_ARCH}-${CT_TARGET_VENDOR:-unknown}-${CT_TARGET_KERNEL}${CT_TARGET_KERNEL:+-}${CT_TARGET_SYS}" + + # Sanity checks + __sed_alias="" + if [ -n "${CT_TARGET_ALIAS_SED_EXPR}" ]; then + __sed_alias=$(echo "${CT_TARGET}" |sed -r -e "${CT_TARGET_ALIAS_SED_EXPR}") + fi + case ":${CT_TARGET_VENDOR}:${CT_TARGET_ALIAS}:${__sed_alias}:" in + :*" "*:*:*:) CT_Abort "Don't use spaces in the vendor string, it breaks things.";; + :*"-"*:*:*:) CT_Abort "Don't use dashes in the vendor string, it breaks things.";; + :*:*" "*:*:) CT_Abort "Don't use spaces in the target alias, it breaks things.";; + :*:*:*" "*:) CT_Abort "Don't use spaces in the target sed transform, it breaks things.";; + esac + + # Canonicalise it + CT_TARGET=$(CT_DoConfigSub "${CT_TARGET}") # Prepare the target CFLAGS CT_ARCH_TARGET_CFLAGS="${CT_ARCH_TARGET_CFLAGS} ${CT_ARCH_ENDIAN_CFLAG}" -- cgit v0.10.2-6-g49f6