typedef onerror default

[Steve Loughran <st...@apache.org>]

Why is the onerror default of typedef "fail" and not "failall"?

The effect is that a typedef like this
     <typedef resource="checkstyletask.properties"
              uri="antlib:com.puppycrawl.checkstyle"
      />

will warn but not fail if the property file is missing. Surely a missing 
declaration file is important enough that the default should be "fail 
the build right now".

proposal
  -we change the "new for ant1.7" default to failall
  -we allow for a failuremessage attr that lets you say "add checkstyle 
to your build via -lib" or whatever.

-steve

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@ant.apache.org
For additional commands, e-mail: dev-help@ant.apache.org

Re: typedef onerror default

[Peter Reilly <pe...@apache.org>]

Steve Loughran wrote:

> Why is the onerror default of typedef "fail" and not "failall"?

It was for BC reasons.
i.e. this is the behaviour for 1.5 when resource was used.

Peter

>
> The effect is that a typedef like this
>     <typedef resource="checkstyletask.properties"
>              uri="antlib:com.puppycrawl.checkstyle"
>      />
>
> will warn but not fail if the property file is missing. Surely a 
> missing declaration file is important enough that the default should 
> be "fail the build right now".
>
> proposal
>  -we change the "new for ant1.7" default to failall
>  -we allow for a failuremessage attr that lets you say "add checkstyle 
> to your build via -lib" or whatever.
>
> -steve
>
> ---------------------------------------------------------------------
> To unsubscribe, e-mail: dev-unsubscribe@ant.apache.org
> For additional commands, e-mail: dev-help@ant.apache.org
>
>
>


---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@ant.apache.org
For additional commands, e-mail: dev-help@ant.apache.org