You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@impala.apache.org by jo...@apache.org on 2022/05/17 02:24:33 UTC
[impala] 02/02: [tools] fix set-classpath for other POSIX shells
This is an automated email from the ASF dual-hosted git repository.
joemcdonnell pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/impala.git
commit 298d064d23b9fc1eb10637406e92809f3f994e92
Author: Michael Smith <mi...@cloudera.com>
AuthorDate: Wed May 11 13:28:35 2022 -0700
[tools] fix set-classpath for other POSIX shells
Updates comparison to use `=` instead of `==` so it works with other
POSIX shells like zsh. `==` is a bash-specific alias for `=`.
Change-Id: Id09ab96a6b1de7cd96eb739221d6aeaa239888d1
Reviewed-on: http://gerrit.cloudera.org:8080/18519
Reviewed-by: Impala Public Jenkins <im...@cloudera.com>
Tested-by: Impala Public Jenkins <im...@cloudera.com>
---
bin/set-classpath.sh | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/bin/set-classpath.sh b/bin/set-classpath.sh
index d15bc00aa..bbdc214bd 100644
--- a/bin/set-classpath.sh
+++ b/bin/set-classpath.sh
@@ -48,7 +48,7 @@ fi
CLASSPATH=$(cat $FE_CP_FILE):"$CLASSPATH"
-if [ "${1:-notest}" == "test" ]; then
+if [[ "${1:-notest}" = "test" ]]; then
FE_TEST_CP_FILE="$IMPALA_HOME/fe/target/test-classpath.txt"
CLASSPATH=$(cat $FE_TEST_CP_FILE):"$CLASSPATH"
fi