ant-user mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Marcus Lindblom <ma...@yar.nu>
Subject Re: Customizable handling of unset properties
Date Wed, 20 Sep 2006 10:55:35 GMT
Ok. Someone had provided a patch in issue 40093, which adresses this, so 
I'll just wait for that one to be merged into ant proper then.

The PropertyHelper class is indeed rather confusing.

Thanks,
/Marcus

Peter Reilly wrote:
> Replacement of propertyhelper is broken in current ant:
> see:
> http://issues.apache.org/bugzilla/show_bug.cgi?id=40093
> and
> http://issues.apache.org/bugzilla/show_bug.cgi?id=25998
> However some people have done it see:
> http://www.efanomars.net/pf/
>
> My feeling that the replacement of propertyhelper was
> always broken and it is a very heavy handed way of
> adding custom property handling.
> Note that PropertyHelper also provides an alternative
> way of adding optional behavour - using hooks. I think
> that this is also broken (I do not understand what it
> is meant to do). PropertyHelper also has "namespace"
> attributes ... which do not get used.
>
>
> Peter
>
>
>
> On 9/20/06, Marcus Lindblom <macke@yar.nu> wrote:
>>
>> Marcus Lindblom wrote:
>> > Dominique Devienne wrote:
>> >>> I am sure other Ant committers will feel differently. I do not see a
>> >>> big urge to implement this.
>> >>>
>> >>> On the other side, why not ?
>> >> [snip}
>> >>
>> > Righto. I'll see what I can come up with then. :)
>> I got response on the bugzilla issue, that there is a PropertyHelper
>> class that I should be able to override to do this. It is, apparently,
>> just a bit awkward to replace the current instance. However, I was
>> informed that it should be doable with a <script> task. Fair enough. :)
>> .. I'm posting here be
>>
>> Has anyone any experience with doing that, because I don't know where to
>> set that class? (I'd assume not, otherwise I would have expected to be
>> pointed there on my first post. :)
>> PropertyHelper.getPropertyHelper(Project p) .. doesn't seem get what I
>> want, and the doc for that function indicates that it's a factyr method,
>> thus creating a new object.
>>
>> Any input?
>>
>> Cheers,
>> /Marcus
>>
>>
>>
>> ---------------------------------------------------------------------
>> To unsubscribe, e-mail: user-unsubscribe@ant.apache.org
>> For additional commands, e-mail: user-help@ant.apache.org
>>
>>
>


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


Mime
View raw message