You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@yetus.apache.org by aw...@apache.org on 2016/04/13 16:17:06 UTC
yetus git commit: YETUS-375. build driver: maven install actually
needs to be dumber
Repository: yetus
Updated Branches:
refs/heads/YETUS-156 2a11f5e81 -> 2d3be4993
YETUS-375. build driver: maven install actually needs to be dumber
Signed-off-by: Allen Wittenauer <aw...@apache.org>
Project: http://git-wip-us.apache.org/repos/asf/yetus/repo
Commit: http://git-wip-us.apache.org/repos/asf/yetus/commit/2d3be499
Tree: http://git-wip-us.apache.org/repos/asf/yetus/tree/2d3be499
Diff: http://git-wip-us.apache.org/repos/asf/yetus/diff/2d3be499
Branch: refs/heads/YETUS-156
Commit: 2d3be4993d658674873d7655cb8d827460f97182
Parents: 2a11f5e
Author: Allen Wittenauer <aw...@apache.org>
Authored: Wed Apr 13 06:56:58 2016 -0700
Committer: Allen Wittenauer <aw...@apache.org>
Committed: Wed Apr 13 07:16:56 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/2d3be499/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 ...