enable spaces in data directory

* pass options as proper bash list items so that even if directories
contain spaces, they will be properly passed on to java - #1623
* ignore empty options
This commit is contained in:
Rots 2019-01-11 21:14:20 +01:00
parent 6dd9f41639
commit 74f6cf21c9

26
refine
View File

@ -93,7 +93,9 @@ EOF
} }
add_option() { add_option() {
OPTS="$OPTS $1" if [ ! -z "$*" ] ; then
OPTS+=("$@")
fi
} }
load_configs() { load_configs() {
@ -497,7 +499,7 @@ ui_test() {
echo "Killing OpenRefine" echo "Killing OpenRefine"
/bin/kill -9 $REFINE_PID /bin/kill -9 $REFINE_PID
echo "Cleaning up" echo "Cleaning up"
rm -rf $REFINE_DATA_DIR rm -rf "$REFINE_DATA_DIR"
} }
server_test() { server_test() {
@ -530,7 +532,7 @@ run() {
fi fi
if [ -d $REFINE_CLASSES_DIR ] ; then if [ -d $REFINE_CLASSES_DIR ] ; then
add_option "-Drefine.autoreload=true -Dbutterfly.autoreload=true" add_option "-Drefine.autoreload=true" "-Dbutterfly.autoreload=true"
fi fi
if [ "$OS" = "macosx" ] ; then if [ "$OS" = "macosx" ] ; then
@ -563,18 +565,18 @@ run() {
CLASSPATH="$REFINE_CLASSES_DIR${SEP}$REFINE_LIB_DIR/*" CLASSPATH="$REFINE_CLASSES_DIR${SEP}$REFINE_LIB_DIR/*"
RUN_CMD="$JAVA -cp $CLASSPATH $OPTS com.google.refine.Refine" RUN_CMD=("$JAVA" -cp "$CLASSPATH" "${OPTS[@]}" "com.google.refine.Refine")
echo "$RUN_CMD" echo "${RUN_CMD[@]}"
#echo "" #echo ""
echo "Starting OpenRefine at 'http://${REFINE_HOST}:${REFINE_PORT}/'" echo "Starting OpenRefine at 'http://${REFINE_HOST}:${REFINE_PORT}/'"
echo "" echo ""
if [ -z "$FORK" ] ; then if [ -z "$FORK" ] ; then
exec $RUN_CMD exec "${RUN_CMD[@]}"
else else
$RUN_CMD & "${RUN_CMD[@]}" &
REFINE_PID="$!" REFINE_PID="$!"
fi fi
} }
@ -595,7 +597,7 @@ broker_run() {
fi fi
if [ -d $REFINE_CLASSES_DIR ] ; then if [ -d $REFINE_CLASSES_DIR ] ; then
add_option "-Drefine.autoreload=true -Dbutterfly.autoreload=true" add_option "-Drefine.autoreload=true" "-Dbutterfly.autoreload=true"
add_option "-Drefine.development=true" add_option "-Drefine.development=true"
fi fi
@ -729,7 +731,7 @@ cd `dirname $0`
# ----- Default values ------------------------------------------ # ----- Default values ------------------------------------------
OPTS="" OPTS=()
# ---- OS-specific support -------------------------------------- # ---- OS-specific support --------------------------------------
@ -786,7 +788,7 @@ while [ $# -ne 0 ] ; do
-k) shift; REFINE_GOOGLE_API_KEY="$1"; shift; continue;; -k) shift; REFINE_GOOGLE_API_KEY="$1"; shift; continue;;
-v) shift; REFINE_VERBOSITY="$1"; shift; continue;; -v) shift; REFINE_VERBOSITY="$1"; shift; continue;;
-x) shift; REFINE_EXTRA_OPTS="$1"; shift; continue;; -x) shift; REFINE_EXTRA_OPTS="$1"; shift; continue;;
--debug) shift; add_option '-Xdebug -Xrunjdwp:transport=dt_socket,address=8000,server=y,suspend=n'; continue;; --debug) shift; add_option '-Xdebug' '-Xrunjdwp:transport=dt_socket,address=8000,server=y,suspend=n'; continue;;
--jmx) shift; add_option '-Dcom.sun.management.jmxremote'; continue;; --jmx) shift; add_option '-Dcom.sun.management.jmxremote'; continue;;
-*) fail "Invalid option: $1";; -*) fail "Invalid option: $1";;
*) break;; *) break;;
@ -806,7 +808,7 @@ fi
if [ -z "$JAVA_OPTIONS" ] ; then if [ -z "$JAVA_OPTIONS" ] ; then
JAVA_OPTIONS="" JAVA_OPTIONS=""
fi fi
add_option "$JAVA_OPTIONS" add_option "${JAVA_OPTIONS[@]}"
if [ -z "$REFINE_MEMORY" ] ; then if [ -z "$REFINE_MEMORY" ] ; then
REFINE_MEMORY="1024M" REFINE_MEMORY="1024M"
@ -814,7 +816,7 @@ fi
if [ -z "$REFINE_MIN_MEMORY" ] ; then if [ -z "$REFINE_MIN_MEMORY" ] ; then
REFINE_MIN_MEMORY="256M" REFINE_MIN_MEMORY="256M"
fi fi
add_option "-Xms$REFINE_MIN_MEMORY -Xmx$REFINE_MEMORY -Drefine.memory=$REFINE_MEMORY" add_option "-Xms$REFINE_MIN_MEMORY" "-Xmx$REFINE_MEMORY" "-Drefine.memory=$REFINE_MEMORY"
freeRam=UNKNOWN freeRam=UNKNOWN
if [ "$OS" = "macosx" ] ; then if [ "$OS" = "macosx" ] ; then