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