[kudu] 02/02: [scripts] fix bogus warning in start_kudu.sh

[al...@apache.org]

This is an automated email from the ASF dual-hosted git repository.

alexey pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/kudu.git

commit fdf83f7624b797f1f63d4bc0c63a2ebb45966d1d
Author: Alexey Serbin <al...@apache.org>
AuthorDate: Thu Dec 2 12:37:58 2021 -0800

    [scripts] fix bogus warning in start_kudu.sh
    
    Prior to this patch, the start_kudu.sh script would issue a warning
    even if $KUDU_HOME/www directory was present when the KUDU_HOME
    environment variable was defined:
    
      Cannot find webroot directory  at $KUDU_HOME/www or $BUILDDIR/../../www
    
    Change-Id: I81a7d1c65c4b2c0b61a56b45901556d4d36a33b6
    Reviewed-on: http://gerrit.cloudera.org:8080/18064
    Tested-by: Kudu Jenkins
    Reviewed-by: Andrew Wong <aw...@cloudera.com>
    Reviewed-by: Attila Bukor <ab...@apache.org>
---
 src/kudu/scripts/start_kudu.sh | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/src/kudu/scripts/start_kudu.sh b/src/kudu/scripts/start_kudu.sh
index cce9316..37cef7f 100755
--- a/src/kudu/scripts/start_kudu.sh
+++ b/src/kudu/scripts/start_kudu.sh
@@ -129,7 +129,7 @@ if [ -z "$KUDU_HOME" ] || [ ! -d "$KUDU_HOME/www" ]; then
   WEBSERVER_DOC_ROOT="$BUILDDIR/../../www"
 fi
 
-if [ ! -d "$WEBSERVER_DOC_ROOT" ]; then
+if [ -n "$WEBSERVER_DOC_ROOT" ] && [ ! -d "$WEBSERVER_DOC_ROOT" ]; then
   echo  -n "Cannot find webroot directory $WEBSERVER_DOC_ROOT at "
   echo "\$KUDU_HOME/www or \$BUILDDIR/../../www"
 fi