You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@sqoop.apache.org by ma...@apache.org on 2016/11/10 14:20:07 UTC

sqoop git commit: SQOOP-3037: Minor convenience feature - add flag to ant test to enable remote debugging

Repository: sqoop
Updated Branches:
  refs/heads/trunk 8e5006084 -> 68f76982a


SQOOP-3037: Minor convenience feature - add flag to ant test to enable remote debugging

(Anna Szonyi via Attila Szabo)


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

Branch: refs/heads/trunk
Commit: 68f76982a01ff2f7c92279df3466466c9cf5d8cc
Parents: 8e50060
Author: Attila Szabo <ma...@apache.org>
Authored: Thu Nov 10 15:19:04 2016 +0100
Committer: Attila Szabo <ma...@apache.org>
Committed: Thu Nov 10 15:19:04 2016 +0100

----------------------------------------------------------------------
 build.xml | 14 +++++++++-----
 1 file changed, 9 insertions(+), 5 deletions(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/sqoop/blob/68f76982/build.xml
----------------------------------------------------------------------
diff --git a/build.xml b/build.xml
index 7f948b3..b1545b6 100644
--- a/build.xml
+++ b/build.xml
@@ -871,12 +871,19 @@
     <antcall target="checkfailure" inheritRefs="true" />
   </target>
 
+  <property name="remoteDebug" value="false"/>
+
   <target name="test-core">
     <!-- inner target only intended to be used via antcall.
          Does not define its dependencies. Should be invoked through the
          'test' target. Does not fail the build if tests fail.
     -->
-
+    <!-- use ant test -DremoteDebug=true to enable remote debugging -->
+    <condition property="remoteDebugJvmArgs"
+               value="-agentlib:jdwp=transport=dt_socket,server=y,suspend=y,address=2601"
+               else="-ea">
+      <istrue value="${remoteDebug}"/>
+    </condition>
     <delete dir="${test.log.dir}"/>
     <mkdir dir="${test.log.dir}"/>
     <delete dir="${build.test}/data"/>
@@ -899,10 +906,7 @@
       <!-- enable asserts in tests -->
       <jvmarg value="-ea" />
 
-      <!-- uncomment this if you want to attach a debugger -->
-      <!--
-      <jvmarg line="-Xdebug -Xrunjdwp:transport=dt_socket,server=y,suspend=y,address=2601" />
-      -->
+      <jvmarg value="${remoteDebugJvmArgs}"/>
 
       <sysproperty key="test.build.data" value="${build.test}/data"/>
       <sysproperty key="build.test" value="${build.test}"/>