antoine 2003/08/29 06:05:06
Modified: docs/manual/CoreTasks java.html
src/main/org/apache/tools/ant/taskdefs Java.java
src/testcases/org/apache/tools/ant/taskdefs JavaTest.java
Log:
Rollback temporarily the setting of new permissions when failonerror="true"
and fork="false"
Revision Changes Path
1.23 +3 -2 ant/docs/manual/CoreTasks/java.html
Index: java.html
===================================================================
RCS file: /home/cvs/ant/docs/manual/CoreTasks/java.html,v
retrieving revision 1.22
retrieving revision 1.23
diff -u -r1.22 -r1.23
--- java.html 28 Aug 2003 13:46:33 -0000 1.22
+++ java.html 29 Aug 2003 13:05:06 -0000 1.23
@@ -88,7 +88,7 @@
<tr>
<td valign="top">failonerror</td>
<td valign="top">Stop the buildprocess if the command exits with a
- returncode other than 0. Default is "false"(see <a
href="#failonerror">note</a>)</td>
+ returncode other than 0. Default is "false"<!--(see <a
href="#failonerror">note)--></a></td>
<td align="center" valign="top">No</td>
</tr>
<tr>
@@ -220,12 +220,13 @@
<p>When the permission RuntimePermission exitVM has not been granted (or has
been revoked) the System.exit() call will be intercepted
and treated like indicated in <i>failonerror</i>.</p>
+<!--
<p><a name="failonerror"/>
If you specify <code>failonerror="true"</code> and you do not
specify permissions,
a set of default permissions will be added to your java invocation making
sure that
a non zero return code will lead to a <code>BuildException</code>.
</p>
-
+-->
<p>Settings will be ignored if fork is enabled.</p>
<p><em>since Ant 1.6</em>.</p>
1.72 +2 -1 ant/src/main/org/apache/tools/ant/taskdefs/Java.java
Index: Java.java
===================================================================
RCS file: /home/cvs/ant/src/main/org/apache/tools/ant/taskdefs/Java.java,v
retrieving revision 1.71
retrieving revision 1.72
diff -u -r1.71 -r1.72
--- Java.java 28 Aug 2003 13:46:33 -0000 1.71
+++ Java.java 29 Aug 2003 13:05:06 -0000 1.72
@@ -180,12 +180,13 @@
log("bootclasspath ignored when same JVM is used.",
Project.MSG_WARN);
}
-
+ /*
if (perm == null && failOnError == true) {
perm = new Permissions();
log("running " + this.cmdl.getClassname()
+ " with default permissions (exit forbidden)",
Project.MSG_VERBOSE);
}
+ */
log("Running in same VM " + cmdl.describeJavaCommand(),
Project.MSG_VERBOSE);
}
1.16 +2 -1
ant/src/testcases/org/apache/tools/ant/taskdefs/JavaTest.java
Index: JavaTest.java
===================================================================
RCS file:
/home/cvs/ant/src/testcases/org/apache/tools/ant/taskdefs/JavaTest.java,v
retrieving revision 1.15
retrieving revision 1.16
diff -u -r1.15 -r1.16
--- JavaTest.java 28 Aug 2003 13:46:33 -0000 1.15
+++ JavaTest.java 29 Aug 2003 13:05:06 -0000 1.16
@@ -195,7 +195,7 @@
executeTarget("testResultPropertyNonZeroNoFork");
assertEquals("-1",project.getProperty("exitcode"));
}
-
+ /*
public void testRunFailWithFailOnError() {
expectBuildExceptionContaining("testRunFailWithFailOnError",
"non zero return code",
@@ -205,6 +205,7 @@
public void testRunSuccessWithFailOnError() {
executeTarget("testRunSuccessWithFailOnError");
}
+ */
public void testSpawn() {
FileUtils fileutils = FileUtils.newFileUtils();
File logFile = fileutils.createTempFile("spawn","log",
project.getBaseDir());
---------------------------------------------------------------------
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]