You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@sqoop.apache.org by ab...@apache.org on 2014/12/12 05:44:41 UTC

sqoop git commit: SQOOP-1858: Sqoop2: Print out warnings from git apply on output of precommit hook

Repository: sqoop
Updated Branches:
  refs/heads/sqoop2 e2e5aa8bf -> c3900910f


SQOOP-1858: Sqoop2: Print out warnings from git apply on output of precommit hook

(Jarek Jarcec Cecho via Abraham Elmahrek)


Project: http://git-wip-us.apache.org/repos/asf/sqoop/repo
Commit: http://git-wip-us.apache.org/repos/asf/sqoop/commit/c3900910
Tree: http://git-wip-us.apache.org/repos/asf/sqoop/tree/c3900910
Diff: http://git-wip-us.apache.org/repos/asf/sqoop/diff/c3900910

Branch: refs/heads/sqoop2
Commit: c3900910f8fd0e86913c3e55e8b59ff7acda2b3f
Parents: e2e5aa8
Author: Abraham Elmahrek <ab...@apache.org>
Authored: Thu Dec 11 22:43:53 2014 -0600
Committer: Abraham Elmahrek <ab...@apache.org>
Committed: Thu Dec 11 22:43:53 2014 -0600

----------------------------------------------------------------------
 dev-support/test-patch.py | 5 +++--
 1 file changed, 3 insertions(+), 2 deletions(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/sqoop/blob/c3900910/dev-support/test-patch.py
----------------------------------------------------------------------
diff --git a/dev-support/test-patch.py b/dev-support/test-patch.py
index e84ec2a..a98dce4 100755
--- a/dev-support/test-patch.py
+++ b/dev-support/test-patch.py
@@ -192,9 +192,10 @@ def git_apply(result, cmd, patch_file, strip, output_dir):
     with open(output_file) as fh:
       output = fh.read()
   if rc == 0:
-    result.success("Patch applied correctly")
     if output:
-      print output
+      result.success("Patch applied, but there has been warnings:\n{code}%s{code}\n" % (output))
+    else:
+      result.success("Patch applied correctly")
   else:
     result.fatal("failed to apply patch (exit code %d):\n{code}%s{code}\n" % (rc, output))