You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@cocoon.apache.org by Stephan Michels <st...@apache.org> on 2003/02/07 14:36:53 UTC

Re: cvs commit: xml-cocoon2 build.xml (Removing warning for transaction (which is not used))

BTW, the slide block can compile with the help of the mock classes,
but doesn't working without a copy of jta.jar, similar to the mail
example.

So, should we really remove the warning?

Stephan Michels.


On 7 Feb 2003 cziegeler@apache.org wrote:

> cziegeler    2003/02/07 05:17:15
>
>   Modified:    .        build.xml
>   Log:
>   Removing warning for transaction (which is not used)
>
>   Revision  Changes    Path
>   1.319     +2 -17     xml-cocoon2/build.xml
>
>   Index: build.xml
>   ===================================================================
>   RCS file: /home/cvs/xml-cocoon2/build.xml,v
>   retrieving revision 1.318
>   retrieving revision 1.319
>   diff -u -r1.318 -r1.319
>   --- build.xml	5 Feb 2003 13:33:47 -0000	1.318
>   +++ build.xml	7 Feb 2003 13:17:14 -0000	1.319
>   @@ -529,10 +529,6 @@
>                   classname="javax.activation.DataSource"/>
>
>        <available classpathref="classpath"
>   -               property="transaction.present"
>   -               classname="javax.transaction.Transaction"/>
>   -
>   -    <available classpathref="classpath"
>                   property="mail.present"
>                   classname="javax.mail.message"/>
>      </target>
>   @@ -812,17 +808,6 @@
>        </antcall>
>      </target>
>
>   -  <!-- Outputs a warning if javax.transaction.* classes are missing during compilation -->
>   -  <target name="transaction-warn" unless="transaction.present" depends="optional-tests">
>   -    <antcall target="op-warning">
>   -      <param name="thing" value="Transaction"/>
>   -      <param name="recovery"
>   -        value="Get the SUN Java Transaction package and place the jar in the lib/local dir"/>
>   -      <param name="message"
>   -             value="The SUN Java Transaction package is required for the Slide classes"/>
>   -    </antcall>
>   -  </target>
>   -


---------------------------------------------------------------------
To unsubscribe, e-mail: cocoon-dev-unsubscribe@xml.apache.org
For additional commands, email: cocoon-dev-help@xml.apache.org


RE: cvs commit: xml-cocoon2 build.xml (Removing warning for transaction (which is not used))

Posted by Carsten Ziegeler <cz...@s-und-n.de>.
But the warning didn't say anything about the block or why it is needed.
So we can either a) make the message more verbose or b) leave it out.
I'm fine with both solutions.

Carsten

> -----Original Message-----
> From: Stephan Michels [mailto:stephan@apache.org]
> Sent: Friday, February 07, 2003 2:37 PM
> To: cocoon-dev@xml.apache.org
> Subject: Re: cvs commit: xml-cocoon2 build.xml (Removing warning for
> transaction (which is not used))
> 
> 
> 
> BTW, the slide block can compile with the help of the mock classes,
> but doesn't working without a copy of jta.jar, similar to the mail
> example.
> 
> So, should we really remove the warning?
> 
> Stephan Michels.
> 
> 
> On 7 Feb 2003 cziegeler@apache.org wrote:
> 
> > cziegeler    2003/02/07 05:17:15
> >
> >   Modified:    .        build.xml
> >   Log:
> >   Removing warning for transaction (which is not used)
> >
> >   Revision  Changes    Path
> >   1.319     +2 -17     xml-cocoon2/build.xml
> >
> >   Index: build.xml
> >   ===================================================================
> >   RCS file: /home/cvs/xml-cocoon2/build.xml,v
> >   retrieving revision 1.318
> >   retrieving revision 1.319
> >   diff -u -r1.318 -r1.319
> >   --- build.xml	5 Feb 2003 13:33:47 -0000	1.318
> >   +++ build.xml	7 Feb 2003 13:17:14 -0000	1.319
> >   @@ -529,10 +529,6 @@
> >                   classname="javax.activation.DataSource"/>
> >
> >        <available classpathref="classpath"
> >   -               property="transaction.present"
> >   -               classname="javax.transaction.Transaction"/>
> >   -
> >   -    <available classpathref="classpath"
> >                   property="mail.present"
> >                   classname="javax.mail.message"/>
> >      </target>
> >   @@ -812,17 +808,6 @@
> >        </antcall>
> >      </target>
> >
> >   -  <!-- Outputs a warning if javax.transaction.* classes are 
> missing during compilation -->
> >   -  <target name="transaction-warn" 
> unless="transaction.present" depends="optional-tests">
> >   -    <antcall target="op-warning">
> >   -      <param name="thing" value="Transaction"/>
> >   -      <param name="recovery"
> >   -        value="Get the SUN Java Transaction package and 
> place the jar in the lib/local dir"/>
> >   -      <param name="message"
> >   -             value="The SUN Java Transaction package is 
> required for the Slide classes"/>
> >   -    </antcall>
> >   -  </target>
> >   -
> 
> 
> ---------------------------------------------------------------------
> To unsubscribe, e-mail: cocoon-dev-unsubscribe@xml.apache.org
> For additional commands, email: cocoon-dev-help@xml.apache.org
> 

---------------------------------------------------------------------
To unsubscribe, e-mail: cocoon-dev-unsubscribe@xml.apache.org
For additional commands, email: cocoon-dev-help@xml.apache.org