From: Peter Rosin Date: Wed, 9 Jan 2013 11:58:55 +0000 (+0100) Subject: tests: fix nits in path handling X-Git-Tag: v2.4.2.418~61 X-Git-Url: http://git.ipfire.org/?a=commitdiff_plain;h=820e373cf2521e553db409e87c4ccc5e6cc4b074;p=thirdparty%2Flibtool.git tests: fix nits in path handling tests/bindir.at: func_save_and_prepend_path is already checking $bindirneeded internally, so remove checks from call sites. Add proper quoting while at it. (func_save_and_prepend_path): Use $PATH_SEPARATOR instead of a literal colon (in both instances). Signed-off-by: Peter Rosin --- diff --git a/tests/bindir.at b/tests/bindir.at index 9786d0826..cc9c24a29 100644 --- a/tests/bindir.at +++ b/tests/bindir.at @@ -82,7 +82,7 @@ func_save_and_prepend_path () { save_PATH=$PATH if $bindirneeded; then - PATH=$1:$PATH + PATH=$1$PATH_SEPARATOR$PATH fi export PATH } @@ -143,7 +143,7 @@ LT_AT_NOINST_EXEC_CHECK([./main]) # Ensure libraries can be found on PATH, if we are on one # of the affected platforms, before testing the shared version. -func_save_and_prepend_path $curdir/$objdir +func_save_and_prepend_path "$curdir/$objdir" $bindirneeded && { LT_AT_NOINST_EXEC_CHECK([$objdir/main]) } @@ -194,7 +194,7 @@ func_save_and_prepend_path () { save_PATH=$PATH if $bindirneeded; then - PATH=$1:$PATH + PATH=$1$PATH_SEPARATOR$PATH fi export PATH } @@ -282,9 +282,7 @@ do fi # And that it can be executed. - extrapath= - $bindirneeded && extrapath=$libdir/../bin - func_save_and_prepend_path $extrapath + func_save_and_prepend_path "$libdir/../bin" LT_AT_EXEC_CHECK([$curdir/sbin/main$EXEEXT], [0], [ignore], [ignore], []) func_restore_path @@ -346,9 +344,7 @@ do fi # And that it can be executed. - extrapath= - $bindirneeded && extrapath=$bindir - func_save_and_prepend_path $extrapath + func_save_and_prepend_path "$bindir" LT_AT_EXEC_CHECK([$curdir/sbin/main$EXEEXT], [0], [ignore], [ignore], []) func_restore_path