ant-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Peter Donald <dona...@apache.org>
Subject Re: [PATCH]Re: if and unless attributes for all Tasks
Date Wed, 16 May 2001 01:32:10 GMT
At 05:30  15/5/01 -0700, Roger Vaughn wrote:
>--- Alan George <alang@hereuare.com> wrote:
>> Roger Vaughn wrote:
>> 
>> > An if/unless uber-task would eliminate the need
>> for
>> > this, but since Ant doesn't (yet?) support nested
>> > tasks, could be difficult to implement.
>> >
>> 
>> I'm not sure what you mean above, but the below
>> patch implements an
>> if/unless for everything that 'isa' Task.
>> 
>> -alan
>
>Not that it matters, but this was what I meant:
>
>  <target name="t1">
>    <if name="some.property">
>      <task1>
>      <task2>
>    </if>
>  </target>
>
>I seriously doubt we'll ever see (need?) nested tasks,
>so I don't expect this to happen.  But since it was so
>easy to write the patch, I don't see why it's a big
>issue.
>
>How about it folks?  It doesn't break your current
>scripts or change the way you work, and you don't have
>to use it if you don't want it, so why not include it?

Sounds like:
Why not add Blah It doesn't break your current
scripts or change the way you work, and you don't have
to use it if you don't want it, so why not include it?

replace Blah with one of:
* conditional operators in if attributes (&&, ||, () etc)
* if/while/for tasks
* if/unless attributes on all tasks
* the ability to have no target definition (just one tasklist)
* some other feature that has been -1'ed in the past and good reasons given
for it

Read the archives to understand why we don't subscribe to that kind of
policy and why that in particular your request will be -1'ed ;)

Cheers,

Pete

*-----------------------------------------------------*
| "Faced with the choice between changing one's mind, |
| and proving that there is no need to do so - almost |
| everyone gets busy on the proof."                   |
|              - John Kenneth Galbraith               |
*-----------------------------------------------------*


Mime
View raw message