GitHub user trystanleftwich opened a pull request:

    https://github.com/apache/spark/pull/21534

    [SPARK-24526][build] Spaces in the build dir causes failures in the 
build/mvn script

    
    ## What changes were proposed in this pull request?
    
    Fix the call to ${MVN_BIN} to be wrapped in quotes so it will handle having 
spaces in the path.
    
    ## How was this patch tested?
    
    Ran the following to confirm using the build/mvn tool with a space in the 
build dir now works without error
    
    ```
    mkdir /tmp/test\ spaces
    cd /tmp/test\ spaces
    git clone https://github.com/apache/spark.git
    cd spark
    # Remove all mvn references in PATH so the script will download mvn to the 
local dir
    ./build/mvn -DskipTests clean package
    ```
    
    Please review http://spark.apache.org/contributing.html before opening a 
pull request.


You can merge this pull request into a Git repository by running:

    $ git pull https://github.com/trystanleftwich/spark SPARK-24526

Alternatively you can review and apply these changes as the patch at:

    https://github.com/apache/spark/pull/21534.patch

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

    This closes #21534
    
----
commit bb12f3e2ad74f9d4c89e1c7adab4d306fa87b101
Author: trystanleftwich <trystan@...>
Date:   2018-06-12T04:44:33Z

    [SPARK-24526][build] Spaces in the build dir causes failures in the 
build/mvn script

----


---

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

Reply via email to