In the maven-tomcat plugin, if we want to precompile JSP, we have to add
@maven.tomcat.generated@ in the web.xml for merging.

But if we set the maven.tomcat.precompile = false and not remove the TAG in
web.xml, it will cause error when startup Tomcat, right ?

So, I suggest to add another condition as following in plugin.jelly.

Eric


plugin.jelly
---------------------------
<goal name="tomcat:mergexml">
    <j:if test="${context.getVariable('maven.tomcat.precompile') =='true'}">
      <loadfile property="maven.tomcat.generated"
srcFile="${maven.build.dir}/tomcat/src/WEB-INF/generated_web.xml"/>

      <copy file="${maven.war.webxml}"
          toFile="${maven.tomcat.webapp.dir}/WEB-INF/web.xml"
          overwrite="true">
        <filterset>
           <filter token="maven.tomcat.generated"
value="${maven.tomcat.generated}"/>
        </filterset>
       </copy>
    </j:if>
   <j:if test="${context.getVariable('maven.tomcat.precompile') =='false'}">
      <copy file="${maven.war.webxml}"
          toFile="${maven.tomcat.webapp.dir}/WEB-INF/web.xml"
          overwrite="true">
        <filterset>
           <filter token="maven.tomcat.generated" value=""/>
        </filterset>
       </copy>
    </j:if>
  </goal>






----- Original Message ----- 
From: "Nathan Coast" <[EMAIL PROTECTED]>
To: "Maven Users" <[EMAIL PROTECTED]>
Sent: Monday, June 07, 2004 9:14 AM
Subject: [ANN] maven-tomcat-plugin 1.1 release


> Hi
>
> the codeczar team is pleased to announce the release of the
> maven-tomcat-plugin 1.1
>
> new features:
> 1) updated dependencies to catalina 5.X classes.
> 2) added Jasper JSP pre-compilation (enabled by default).
> 3) added support for Tomcat context descriptors.
> 4) added an example project.
>
> project homepage
> http://www.codeczar.com/products/maven-tomcat-plugin/index.html
> download & installation instructions
> http://www.codeczar.com/products/maven-tomcat-plugin/download.html
>
> cheers
> Nathan
>
>
> ---------------------------------------------------------------------
> 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