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