You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@drill.apache.org by cg...@apache.org on 2020/10/01 19:41:15 UTC

[drill] branch master updated: Fix syntax error in "drill-config.sh"

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

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


The following commit(s) were added to refs/heads/master by this push:
     new 2eb6979  Fix syntax error in "drill-config.sh"
2eb6979 is described below

commit 2eb6979206ad1b6f830a070dffa822edbfac6189
Author: Gareth Western <ga...@garethwestern.com>
AuthorDate: Mon Sep 21 11:46:11 2020 +0200

    Fix syntax error in "drill-config.sh"
    
    The drill-config.sh has a syntax error on line 144 with the usage of "let". The command following let should be enclosed in quotes, otherwise the following syntax error occurs:
    /home/someone/proj/apache-drill-1.17.0/bin/drill-config.sh: line 144: let: lineCount=: syntax error: operand expected (error token is "=")
---
 distribution/src/main/resources/drill-config.sh | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/distribution/src/main/resources/drill-config.sh b/distribution/src/main/resources/drill-config.sh
index 65a85c0..d8aa085 100644
--- a/distribution/src/main/resources/drill-config.sh
+++ b/distribution/src/main/resources/drill-config.sh
@@ -141,7 +141,7 @@ function checkExecutableLineCount() {
   #Not valid file
   if [ ! -f "$1" ]; then echo $lineCount; return; fi
   # Filtering out commented and empty lines
-  let lineCount=`cat $1 | grep -v '#' | grep -v ^$ | wc -l`
+  let "lineCount=`cat $1 | grep -v '#' | grep -v ^$ | wc -l`"
   echo $lineCount
   return
 }