ant-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Erik Hatcher" <>
Subject Re: cvs commit: jakarta-ant/src/testcases/org/apache/tools/ant/taskdefs/optional
Date Fri, 22 Feb 2002 02:06:12 GMT
Ok Diane... have at it!

Let me know if I break any of your tests with <propertyfile>.  Also, perhaps
now is the time to learn how to integrate with the builtin test cases.  To
run this particular one, after bootstrapping, run this:


This particular test case is a bit confusing because of some indirection its
doing: it writes two property files, one read in by the build file, the
other is manipulated by <propertyfile> and therefore sets up some existing
values to test against, and then there is the <property file="..."/> in the
'exercise' target that loads the property file just saved which provides the
PropertyFileTest code with the properties to easily compare against expected

If you find a flaw, the best way to let me know is to update the test case
to demonstrate it. If you don't want to attempt a fix, just send me the
patch and I'll take care of it.  (don't check a broken test case back in!

Also, (and why does this happen right when you click to commit a change?) on
second thought, I think I'll add the "never" capability back in to simply
blank a date type property which seemed to be the original intent. It would
really only make sense as a default value though.  Thoughts one way or

Any idea what the "now"/"never" on the operations was all about?


----- Original Message -----
From: <>
To: <>
Sent: Thursday, February 21, 2002 8:53 PM
Subject: cvs commit:

> ehatcher    02/02/21 17:53:36
>   Modified:    docs/manual/OptionalTasks propertyfile.html
>                src/main/org/apache/tools/ant/taskdefs/optional
>                src/etc/testcases/taskdefs/optional propertyfile.xml
>                src/testcases/org/apache/tools/ant/taskdefs/optional
>   Log:
>   Here goes: propertyfile refactoring to hopefully fix all outstanding
issues with it. "now" and "never" were removed as operations (didn't make
sense). "never" was removed as a value (it caused a null pointer exception
>   Other code cleanup (axing "m_" prefixes).
>   Added "unit" attribute to <entry> defaulting to DAY. This was
essentially the previous default, but it was far too confusing.
>   Test cases beefed up to consider several scenarios.
>   There may be more changes needed to satisfy other use cases, and
certainly some previous (undocumented) behavior has been changed (for the
>   Is there a use-case for "never" as a value?

To unsubscribe, e-mail:   <>
For additional commands, e-mail: <>

View raw message