YETUS-375. build driver: maven install actually needs to be dumber 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/dfb087c2 Tree: http://git-wip-us.apache.org/repos/asf/yetus/tree/dfb087c2 Diff: http://git-wip-us.apache.org/repos/asf/yetus/diff/dfb087c2 Branch: refs/heads/YETUS-156 Commit: dfb087c2317ef0cf1363890afa1a73ebcd2f627a Parents: 0f5c6b5 Author: Allen Wittenauer <[email protected]> Authored: Wed Apr 13 06:56:58 2016 -0700 Committer: Allen Wittenauer <[email protected]> Committed: Sun Apr 17 14:07:17 2016 -0700 ---------------------------------------------------------------------- precommit/test-patch.d/maven.sh | 9 +-------- 1 file changed, 1 insertion(+), 8 deletions(-) ---------------------------------------------------------------------- http://git-wip-us.apache.org/repos/asf/yetus/blob/dfb087c2/precommit/test-patch.d/maven.sh ---------------------------------------------------------------------- diff --git a/precommit/test-patch.d/maven.sh b/precommit/test-patch.d/maven.sh index d65aeaa..3d37c37 100755 --- a/precommit/test-patch.d/maven.sh +++ b/precommit/test-patch.d/maven.sh @@ -523,14 +523,7 @@ function maven_precompile fi if verify_needed_test javac; then - # 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 + need=true else # not everything needs a maven install # but quite a few do ...
