ant-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Burgess, Benjamin" <>
Subject RE: enhancement for PropertyFile, have a patch
Date Fri, 02 Sep 2005 14:06:05 GMT
+1 - excellent proposal

-----Original Message-----
From: Antoine Levy-Lambert [] 
Sent: Friday, September 02, 2005 10:03 AM
To: Ant Developers List
Subject: Re: enhancement for PropertyFile, have a patch

Hello Phil,
I wonder whether, instead of patching PropertyFile, you should not study
how the <filterchain/> framework works and implement the functionalitiy
of propertyfile
as a filter. Then it should be possible to change the implementation of
propertyfile to make it use filters.
To my opinion, all tasks which we have which are manipulating text files
should be rewritten around filters. This would avoid code duplication.
And make the functionality of property file available on the fly.



Phil Hourihane wrote:

>Hi folks,
>I hope I am getting the etiquette right for this; I am
>a new body on the list.
>The PropertyFile (optional) task allows a script to
>alter an existing properties file, but all comments
>and layout are lost. This doesn't get a mention in
>bugzilla. I have an enhancement which retains layout
>and comments when writing a properties file, with
>accompanying tests.
>I have a patch for the affected files, and there are a
>few new files (util class, test class, and test data)
>as well.
>Do the group want the enhancement? Or more detail from
>me? Shall I post the patch and new files? Any feedback
>is welcome.
>Philip Hourihane

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

This message, including any attachments, contains confidential information intended for a
specific individual and purpose, and is protected by law.  If you are not the intended recipient,
please contact sender immediately by reply e-mail and destroy all copies.  You are hereby
notified that any disclosure, copying, or distribution of this message, or the taking of any
action based on it, is strictly prohibited.

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

View raw message