this patch for the jetty maven plugin allows for jsp compilation. tools.jar is added to the classpath and because of some dependency ant.jar as well.

this patch was created from cvs head on 12-22-2004

this is a precompiled version of the modifed jetty plugin. replace this file with the 1.1 version in maven/plugins

Hi Peter,

I was looking into applying your patch but I'm confused. Your latest comment suggest to use some jar containing source files. However, in order to apply your pach I would need the changes in pach format (diff -u).

Thanks a lot

There are two attachments.

#1 The patch file for the changes to allow jsp compilation.

#2 A compiled version of the plugin in case someone experienced the bug and wanted to simply wanted to replace the jetty plugin in their maven installation.

Peter

PS I noticed in the patch that I forgot to remove an unnecessary echo statement. After you apply the patch please remove the line which was used for testing:

<echo message="${tools.jar}"/>

Ok, got it. Thanks Peter.

Just a question. Where do you get the ${tools.jar} value from (who sets it)?

tools.jar is set in maven's bat/shell script

i just reach to the same solution. Others plugins uses ${tools.jar}

juan@dinah jetty $ find ~/.maven/cache/|xargs grep tools.jar
/home/juan/.maven/cache/maven-dotuml-plugin-1.0/plugin.jelly: <ant:path location="${tools.jar}"/>
/home/juan/.maven/cache/maven-dotuml-plugin-1.0-java5-zauber/plugin.jelly: <ant:path location="${tools.jar}"/>
/home/juan/.maven/cache/maven-jellydoc-plugin-1.3.1/project.properties:maven.jar.javadoc=${tools.jar}

Applied (at last!). Sorry for the delay...