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/c9754c0a Tree: http://git-wip-us.apache.org/repos/asf/yetus/tree/c9754c0a Diff: http://git-wip-us.apache.org/repos/asf/yetus/diff/c9754c0a Branch: refs/heads/YETUS-156 Commit: c9754c0a0a5db41152cfe998a6662e1fa08a93ee Parents: dc96393 Author: Allen Wittenauer <[email protected]> Authored: Tue Apr 12 13:52:33 2016 -0700 Committer: Allen Wittenauer <[email protected]> Committed: Thu Apr 14 00:24:45 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/c9754c0a/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
