I would perfer to wait until after ant 1.7.1 before getting antlib.xml
loaded for ant.

Peter
On 5/22/07, Kevin Jackson <[EMAIL PROTECTED]> wrote:
Hi Matt,

> > > we already had this covered, e.g.:
> > >
> > > <resourcecount count="1">
> > >   <restrict>
> > >     <file file="${file}" />
> > >     <contains text="text" casesensitive="false"
> > >
> > >
> >
> xmlns="antlib:org.apache.tools.ant.types.resources.selectors"
> > > />
> > >   </restrict>
> > > </resourcecount>
> > >
> > > Note that this approach supports any resource type
> > > right off the bat.  Actually with the suggested
> > "add"
> > > idiom, <resourcecontains> should be rewritten as a
> > > macrodef.  :|

I think that using a macrodef makes more sense than using Java in this case

> Back to this... do we plan to replace
> au:assertResourceContains with some usage of the
> <contains> selector as I demonstrated, then remove the
> ResourceContains java condition?

Putting the macrodef in the antunit antlib.xml is trivial, but how do
we make resourcecontains available beyond the scope of antunit?
Should we make resourcecontains a general condition (as it is right
now)?

me:
+1 use a macrodef (less code to maintain)
+1 make it available in ant core, not just in the antunit antlib

Kev

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



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

Reply via email to