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/aa0bd723 Tree: http://git-wip-us.apache.org/repos/asf/yetus/tree/aa0bd723 Diff: http://git-wip-us.apache.org/repos/asf/yetus/diff/aa0bd723 Branch: refs/heads/YETUS-156 Commit: aa0bd7233c23bcb8450ba650eb458ba4c7435f46 Parents: 830620a Author: Allen Wittenauer <[email protected]> Authored: Wed Apr 13 06:56:58 2016 -0700 Committer: Allen Wittenauer <[email protected]> Committed: Fri Apr 22 13:25:07 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/aa0bd723/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 ...
