RE: cvs commit: jakarta-ant/src/main/org/apache/tools/ant NoExitS ecurityManager.java Main.java ExitException.java

[Stephane Bailliez <sb...@imediation.com>]

> -----Original Message-----
> From: Peter Donald [mailto:peter@apache.org]

> I just reverted this because it didn't actually work. All 
> successful builds 
> printed out a stack trace and a error message because 
> System.exit(0) was 
> raising an exception. You really should run ant before you 
> check in things 
> like this ;)

Guilty.

It's hard to figure out that it can go wrong even when modifying one line of
code.
I assumed too much.

[...]
> We can't justifiably implement this in Ant1.x unless we also 
> implemented 
> Conors launcher but that would break backwards compatability 
> anyways. Any 
> task that can potentially call exit should just be forked.

Will revert related changes.

Thanks.

Stephane

--
To unsubscribe, e-mail:   <ma...@jakarta.apache.org>
For additional commands, e-mail: <ma...@jakarta.apache.org>