yetus-commits mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From a.@apache.org
Subject [19/23] yetus git commit: YETUS-373. build driver: maven install should be smarter
Date Sun, 17 Apr 2016 17:15:07 GMT
YETUS-373. build driver: maven install should be smarter

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/bcbeae50
Tree: http://git-wip-us.apache.org/repos/asf/yetus/tree/bcbeae50
Diff: http://git-wip-us.apache.org/repos/asf/yetus/diff/bcbeae50

Branch: refs/heads/YETUS-156
Commit: bcbeae50678a7250d3cd9743548f742ab9ee0e03
Parents: ac9c62e
Author: Allen Wittenauer <aw@apache.org>
Authored: Tue Apr 12 13:52:33 2016 -0700
Committer: Allen Wittenauer <aw@apache.org>
Committed: Sun Apr 17 10:14:34 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/bcbeae50/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


Mime
View raw message