DO NOT REPLY TO THIS EMAIL, BUT PLEASE POST YOUR BUG 
RELATED COMMENTS THROUGH THE WEB INTERFACE AVAILABLE AT
<http://nagoya.apache.org/bugzilla/show_bug.cgi?id=23942>.
ANY REPLY MADE TO THIS MESSAGE WILL NOT BE COLLECTED AND 
INSERTED IN THE BUG DATABASE.

http://nagoya.apache.org/bugzilla/show_bug.cgi?id=23942

Addition of a local property





------- Additional Comments From [EMAIL PROTECTED]  2003-11-23 06:38 -------
Just wanted to mention that the patch (9164) fails on defaults.properties when
patching against the 1.6 Branch.  Version 1.154 is used to patch against, but it
isn't tagged with the ANT_16_BRANCH tag and the one on the 1.6 branch doesn't
define the "nice" task so <patch> returns an error for it and skips applying the
patch.  Everything else seems to work ok.

Oh, one other thing.  In order to compile Local.java, I had to modify the
following from this:
PropertyHelper.getPropertyHelper(
            getProject()).addLocalProperty(name, value);

to this:
PropertyHelper.getPropertyHelper(
            getProject()).addLocalProperty(name, value, false);

The reason is that Local seems to have been built against the older patch (8631
or 8652) where the addLocalProperty() method took a String and an Object, but
not also a boolean as the newest patch adds.

After all that, I built the modified 1.6 Branch and tested my <macrodef> that
sets a property, but uses <local> to keep the property local.  It works 
great!!!!

Is this going to get in soon?  It would be nice to play with a more officially
applied patch and get other users to test it out before a 1.6beta3 release.

Jake

---------------------------------------------------------------------
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]

Reply via email to