YETUS-373. build driver: maven install should be smarter Signed-off-by: Allen Wittenauer <[email protected]>
Project: http://git-wip-us.apache.org/repos/asf/yetus/repo Commit: http://git-wip-us.apache.org/repos/asf/yetus/commit/0c887e48 Tree: http://git-wip-us.apache.org/repos/asf/yetus/tree/0c887e48 Diff: http://git-wip-us.apache.org/repos/asf/yetus/diff/0c887e48 Branch: refs/heads/YETUS-156 Commit: 0c887e484f6c58361dd6145d52eba6854c4c041d Parents: f89eab9 Author: Allen Wittenauer <[email protected]> Authored: Tue Apr 12 13:52:33 2016 -0700 Committer: Allen Wittenauer <[email protected]> Committed: Fri Apr 22 13:25:07 2016 -0700 ---------------------------------------------------------------------- precommit/test-patch.d/maven.sh | 13 ++++++++----- 1 file changed, 8 insertions(+), 5 deletions(-) ---------------------------------------------------------------------- http://git-wip-us.apache.org/repos/asf/yetus/blob/0c887e48/precommit/test-patch.d/maven.sh ---------------------------------------------------------------------- diff --git a/precommit/test-patch.d/maven.sh b/precommit/test-patch.d/maven.sh index 77aab32..d65aeaa 100755 --- a/precommit/test-patch.d/maven.sh +++ b/precommit/test-patch.d/maven.sh @@ -523,7 +523,14 @@ function maven_precompile fi if verify_needed_test javac; then - need=true + # if we are in full build mode, then + # there is no need to do an initial install + # since we will be doing a full compile from basedir + # for projects like hadoop, this saves quite a bit + # of time + if [[ "${BUILDMODE}" = patch ]]; then + need=true + fi else # not everything needs a maven install # but quite a few do ... @@ -542,10 +549,6 @@ function maven_precompile return 0 fi - if [[ "${BUILDMODE}" = Full ]]; then - return 0 - fi - if [[ "${repostatus}" = branch ]]; then big_console_header "maven install: ${PATCH_BRANCH}" else
