Thanks Chris! I did not know the <istrue> condition...

-Vincent

> -----Original Message-----
> From: Christopher Lenz [mailto:[EMAIL PROTECTED]
> Sent: 23 March 2003 18:42
> To: Cactus Developers List
> Subject: Re: cvs commit:
jakarta-cactus/integration/ant/src/scripts/share
> build-share.xml
> 
> [EMAIL PROTECTED] wrote:
> [snip]
> >   +        <!-- Decides whether to halt the build if errors or
failures
> have
> >   +           occurred -->
> >   +
> >   +        <condition property="cactus.test.error.shouldstop">
> >   +          <and>
> >   +            <isset property="cactus.junit.error"/>
> >   +            <equals arg1="${cactus.halt.on.error}" arg2="true"
> trim="true"
> >   +              casesensitive="false"/>
> 
> Why not use <istrue> here? That would also support the 'yes' and 'on'
> values, besides being cleaner.
> 
> --
> Christopher Lenz
> /=/ cmlenz at gmx.de
> 
> 
> ---------------------------------------------------------------------
> 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