diff options
author | Dongjoon Hyun <dongjoon@apache.org> | 2016-04-27 20:56:23 +0100 |
---|---|---|
committer | Sean Owen <sowen@cloudera.com> | 2016-04-27 20:56:23 +0100 |
commit | f405de87c878c49b17acb2c874be1084465384e9 (patch) | |
tree | 39c5e9eda5057d91f326477540086f3908037d06 /.gitignore | |
parent | af92299fdb8d358bbfeb2bf0d88ef7da8e2144b9 (diff) | |
download | spark-f405de87c878c49b17acb2c874be1084465384e9.tar.gz spark-f405de87c878c49b17acb2c874be1084465384e9.tar.bz2 spark-f405de87c878c49b17acb2c874be1084465384e9.zip |
[SPARK-14867][BUILD] Remove `--force` option in `build/mvn`
## What changes were proposed in this pull request?
Currently, `build/mvn` provides a convenient option, `--force`, in order to use the recommended version of maven without changing PATH environment variable. However, there were two problems.
- `dev/lint-java` does not use the newly installed maven.
```bash
$ ./build/mvn --force clean
$ ./dev/lint-java
Using `mvn` from path: /usr/local/bin/mvn
```
- It's not easy to type `--force` option always.
If '--force' option is used once, we had better prefer the installed maven recommended by Spark.
This PR makes `build/mvn` check the existence of maven installed by `--force` option first.
According to the comments, this PR aims to the followings:
- Detect the maven version from `pom.xml`.
- Install maven if there is no or old maven.
- Remove `--force` option.
## How was this patch tested?
Manual.
```bash
$ ./build/mvn --force clean
$ ./dev/lint-java
Using `mvn` from path: /Users/dongjoon/spark/build/apache-maven-3.3.9/bin/mvn
...
$ rm -rf ./build/apache-maven-3.3.9/
$ ./dev/lint-java
Using `mvn` from path: /usr/local/bin/mvn
```
Author: Dongjoon Hyun <dongjoon@apache.org>
Closes #12631 from dongjoon-hyun/SPARK-14867.
Diffstat (limited to '.gitignore')
0 files changed, 0 insertions, 0 deletions