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/12 22:54:45 UTC
[1/2] yetus git commit: YETUS-372. build driver: unit tests still
mentions patches
Repository: yetus
Updated Branches:
refs/heads/YETUS-156 902f49318 -> 9dfaa98fa
YETUS-372. build driver: unit tests still mentions patches
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/68a08a3c
Tree: http://git-wip-us.apache.org/repos/asf/yetus/tree/68a08a3c
Diff: http://git-wip-us.apache.org/repos/asf/yetus/diff/68a08a3c
Branch: refs/heads/YETUS-156
Commit: 68a08a3c24cb2c426aa62e8c48be0f8801171114
Parents: 902f493
Author: Allen Wittenauer <aw...@apache.org>
Authored: Tue Apr 12 13:50:54 2016 -0700
Committer: Allen Wittenauer <aw...@apache.org>
Committed: Tue Apr 12 13:51:06 2016 -0700
----------------------------------------------------------------------
precommit/test-patch.sh | 10 ++++++----
1 file changed, 6 insertions(+), 4 deletions(-)
----------------------------------------------------------------------
http://git-wip-us.apache.org/repos/asf/yetus/blob/68a08a3c/precommit/test-patch.sh
----------------------------------------------------------------------
diff --git a/precommit/test-patch.sh b/precommit/test-patch.sh
index 9f866ae..9fb26dc 100755
--- a/precommit/test-patch.sh
+++ b/precommit/test-patch.sh
@@ -1780,10 +1780,10 @@ function modules_messages
declare statusjdk
declare multijdkmode=false
- if [[ "${repostatus}" == branch ]]; then
- repo=${PATCH_BRANCH}
- elif [[ "${BUILDMODE}" == full ]]; then
+ if [[ "${BUILDMODE}" == full ]]; then
repo="the source"
+ elif [[ "${repostatus}" == branch ]]; then
+ repo=${PATCH_BRANCH}
else
repo="the patch"
fi
@@ -1899,7 +1899,9 @@ function modules_workers
declare result=0
declare argv
- if [[ ${repostatus} == branch ]]; then
+ if [[ "${BUILDMODE}" = full ]]; then
+ repo="the source"
+ elif [[ ${repostatus} == branch ]]; then
repo=${PATCH_BRANCH}
else
repo="the patch"
[2/2] yetus git commit: YETUS-373. build driver: maven install should
be smarter
Posted by aw...@apache.org.
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/9dfaa98f
Tree: http://git-wip-us.apache.org/repos/asf/yetus/tree/9dfaa98f
Diff: http://git-wip-us.apache.org/repos/asf/yetus/diff/9dfaa98f
Branch: refs/heads/YETUS-156
Commit: 9dfaa98fab969cfaf334e65a244d261d9916ca63
Parents: 68a08a3
Author: Allen Wittenauer <aw...@apache.org>
Authored: Tue Apr 12 13:52:33 2016 -0700
Committer: Allen Wittenauer <aw...@apache.org>
Committed: Tue Apr 12 13:52:36 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/9dfaa98f/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