Github user srowen commented on a diff in the pull request:
https://github.com/apache/spark/pull/22333#discussion_r215115678
--- Diff: build/mvn ---
@@ -91,15 +92,23 @@ install_mvn() {
# Install zinc under the build/ folder
install_zinc() {
- local zinc_path="zinc-0.3.15/bin/zinc"
- [ ! -f "${_DIR}/${zinc_path}" ] && ZINC_INSTALL_FLAG=1
- local TYPESAFE_MIRROR=${TYPESAFE_MIRROR:-https://downloads.lightbend.com}
+ ZINC_VERSION=0.3.15
+ ZINC_BIN="$(command -v zinc)"
+ if [ "$ZINC_BIN" ]; then
+ local ZINC_DETECTED_VERSION="$(zinc -version | head -n1 | awk '{print
$5}')"
+ fi
- install_app \
- "${TYPESAFE_MIRROR}/zinc/0.3.15" \
- "zinc-0.3.15.tgz" \
- "${zinc_path}"
- ZINC_BIN="${_DIR}/${zinc_path}"
+ if [ $(version $ZINC_DETECTED_VERSION) -lt $(version $ZINC_VERSION) ];
then
--- End diff --
I assume this evaluates to false if no zinc was detected; that seems to be
how the maven logic above works. OK. I also would like to use my local zinc if
available.
---
---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org
