hopefully better handling of users with Java installed in dirs with spaces on win #358

This commit is contained in:
soulgalore 2014-02-03 21:22:41 +01:00
parent c1f71f3646
commit e06742bf29
1 changed files with 2 additions and 2 deletions

View File

@ -464,7 +464,7 @@ fi
# Log the versions
#*******************************************************
function log_versions {
browserTimeVersion=$($BROWSERTIME -V)
browserTimeVersion="$($BROWSERTIME -V)"
echo "Using sitespeed.io version $SITESPEED_VERSION"
echo "Using PhantomJS version $(phantomjs --version)"
echo "Using Java version $jVersion"
@ -880,7 +880,7 @@ do
local pagefilename=$(get_filename $url $runs)
echo "Collecting Navigation Timing metrics (${BROWSERS_ARRAY[i]}): $url"
## Extra info: If you are using the Chrome driver on OS X, it always output "Starting the ChromeDriver ..." and this is a hack to remove that
($BROWSERTIME --compact --raw -b ${BROWSERS_ARRAY[i]} $PROXY_BROWSER_TIME -n $NUMBER_OF_RUNS -o "$REPORT_DATA_METRICS_DIR/${BROWSERS_ARRAY[i]}/$pagefilename.xml" -ua "\"$USER_AGENT\"" -w $VIEWPORT "$url" 3>&1 1>&2 2>&3 | grep -v '^Starting' ) 3>&1 1>&2 2>&3 >> $REPORT_DATA_DIR/error.log
( $BROWSERTIME --compact --raw -b ${BROWSERS_ARRAY[i]} $PROXY_BROWSER_TIME -n $NUMBER_OF_RUNS -o "$REPORT_DATA_METRICS_DIR/${BROWSERS_ARRAY[i]}/$pagefilename.xml" -ua "\"$USER_AGENT\"" -w $VIEWPORT "$url" 3>&1 1>&2 2>&3 | grep -v '^Starting' ) 3>&1 1>&2 2>&3 >> $REPORT_DATA_DIR/error.log
## If BrowserTime fails, an empty file is created, so remove it
local btSize=$(du -k "$REPORT_DATA_METRICS_DIR/${BROWSERS_ARRAY[i]}/$pagefilename.xml" | cut -f1)