Re: [U-Boot] [PATCH] MAKEALL: Fix case substitution for old bash

2013-03-27 Thread Allen Martin 
On Fri, Mar 22, 2013 at 10:37:03AM -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 york...@freescale.com

Acked-by: Allen Martin amar...@nvidia.com



 ---
 Removed RFC from subject line.
 
  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:]'`
   if [ ${cross_toolchain} ] ; then
   MAKE=make CROSS_COMPILE=${cross_toolchain}
   elif [ ${CROSS_COMPILE} ] ; then
 -- 
 1.7.9.5
 
 

-- 
nvpublic
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot

[U-Boot] [PATCH] MAKEALL: Fix case substitution for old bash

2013-03-22 Thread York Sun 
Bash ver 3.x doesn't support the parameter expansion with case
substitution. Use tr instead.

Signed-off-by: York Sun york...@freescale.com
---
Removed RFC from subject line.

 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:]'`
if [ ${cross_toolchain} ] ; then
MAKE=make CROSS_COMPILE=${cross_toolchain}
elif [ ${CROSS_COMPILE} ] ; then
-- 
1.7.9.5


___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot