From 9f107b6a01e6ee49841e5527b68f58750c4c920e Mon Sep 17 00:00:00 2001 From: Mark Olesen Date: Mon, 6 Oct 2008 10:05:02 +0200 Subject: [PATCH] settings.sh - bugfix - let _foamAddPath and _foamAddLib handle ':'-separated variables This unfortunately gets the order backwards, but at least doesn't try making really stupid directory names TODO: - settings.csh still has the same bug - perhaps _foamAddExistingLib function (or with a switch) to avoid the mkdir --- etc/settings.sh | 45 +++++++++++++++++++++++++++++++-------------- 1 file changed, 31 insertions(+), 14 deletions(-) diff --git a/etc/settings.sh b/etc/settings.sh index 743d8e1fc0..6c05f9f44e 100644 --- a/etc/settings.sh +++ b/etc/settings.sh @@ -33,22 +33,40 @@ _foamAddPath() { - while [ $# -ge 1 ] - do - [ -d $1 ] || mkdir -p $1 - export PATH=$1:$PATH - shift - done + if [ $# -eq 1 ] + then + oldIFS="$IFS" + IFS=':' # split on ':' + set -- $1 + IFS="$oldIFS" + unset oldIFS + fi + + while [ $# -ge 1 ] + do + [ -d $1 ] || mkdir -p $1 + export PATH=$1:$PATH + shift + done } _foamAddLib() { - while [ $# -ge 1 ] - do - [ -d $1 ] || mkdir -p $1 - export LD_LIBRARY_PATH=$1:$LD_LIBRARY_PATH - shift - done + if [ $# -eq 1 ] + then + oldIFS="$IFS" + IFS=':' # split on ':' + set -- $1 + IFS="$oldIFS" + unset oldIFS + fi + + while [ $# -ge 1 ] + do + [ -d $1 ] || mkdir -p $1 + export LD_LIBRARY_PATH=$1:$LD_LIBRARY_PATH + shift + done } @@ -99,7 +117,6 @@ case "$WM_COMPILER_INST" in OpenFOAM) case "$WM_COMPILER" in Gcc) - #export WM_COMPILER_DIR=$WM_THIRD_PARTY_DIR/gcc-4.3.1/platforms/$WM_ARCH$WM_COMPILER_ARCH export WM_COMPILER_DIR=$WM_THIRD_PARTY_DIR/gcc-4.3.2/platforms/$WM_ARCH$WM_COMPILER_ARCH _foamAddLib $WM_THIRD_PARTY_DIR/mpfr-2.3.2/platforms/$WM_ARCH$WM_COMPILER_ARCH/lib _foamAddLib $WM_THIRD_PARTY_DIR/gmp-4.2.3/platforms/$WM_ARCH$WM_COMPILER_ARCH/lib @@ -127,7 +144,7 @@ esac if [ -d "$WM_COMPILER_BIN" ]; then _foamAddPath $WM_COMPILER_BIN - _foamAddLib $WM_COMPILER_LIB + _foamAddLib $WM_COMPILER_LIB fi unset WM_COMPILER_BIN WM_COMPILER_LIB