caoliang-web commented on code in PR #45:
URL: 
https://github.com/apache/doris-spark-connector/pull/45#discussion_r934325715


##########
spark-doris-connector/build.sh:
##########
@@ -45,6 +45,17 @@ usage() {
   exit 1
 }
 
+# we use GNU enhanced version getopt command here for long option names, 
rather than the original version
+# determine the version of the getopt command before using
+getopt -T > /dev/null
+if [ $? -ne 4 ]; then
+  echo "
+    The GNU version of getopt command is required.
+    On Mac OS, you can use Homebrew to install gnu-getopt: brew install 
gnu-getopt, then set gnu-getopt as default getopt. Refernence: 
https://stackoverflow.com/questions/12152077/how-can-i-make-bash-deal-with-long-param-using-getopt-command-in-mac
+  "
+  exit 1
+fi
+

Review Comment:
   The linux environment is possible, and the mac environment executes the 
build command and directly echo 'The GNU version of getopt command is 
required.' The program ends



-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: commits-unsubscr...@doris.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscr...@doris.apache.org
For additional commands, e-mail: commits-h...@doris.apache.org

Reply via email to