On Thu, Mar 21, 2013 at 12:58:15PM -0700, York Sun wrote:
Bash ver 3.x doesn't support the parameter expansion with case substitution. Use tr instead.
Signed-off-by: York Sun yorksun@freescale.com
I am not sure if using 'tr' is a good idea. Any suggestion is welcomed.
MAKEALL | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/MAKEALL b/MAKEALL index c1d8957..ac92ef6 100755 --- a/MAKEALL +++ b/MAKEALL @@ -664,7 +664,7 @@ build_target() { export BUILD_DIR="${output_dir}"
target_arch=$(get_target_arch ${target})
- eval cross_toolchain=$CROSS_COMPILE_${target_arch^^}
- eval cross_toolchain=$CROSS_COMPILE_`echo $target_arch | tr '[:lower:]' '[:upper:]'`
Looks fine to me. I'm not sure if there's a preference between using backtick or $() for shell commands, it looks like both are used in the script.
if [ "${cross_toolchain}" ] ; then MAKE="make CROSS_COMPILE=${cross_toolchain}" elif [ "${CROSS_COMPILE}" ] ; then -- 1.7.9.5
PS: What happened with your bool fixup patch? I'm a big fan of that patch :^)
-Allen