yetus-commits mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From a.@apache.org
Subject [12/19] yetus git commit: YETUS-375. build driver: maven install actually needs to be dumber
Date Sun, 17 Apr 2016 21:07:40 GMT
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/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 <aw@apache.org>
Authored: Wed Apr 13 06:56:58 2016 -0700
Committer: Allen Wittenauer <aw@apache.org>
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 ...


Mime
View raw message