Summary: | classpath inconsistent between Windows and Unix for tools.jar | ||
---|---|---|---|
Product: | Tomcat 5 | Reporter: | mschulz |
Component: | Catalina | Assignee: | Tomcat Developers Mailing List <dev> |
Status: | RESOLVED FIXED | ||
Severity: | normal | ||
Priority: | P2 | ||
Version: | 5.5.15 | ||
Target Milestone: | --- | ||
Hardware: | Sun | ||
OS: | Solaris | ||
Attachments: |
patch for setclasspath.bat to make it consistent with the .sh version
Patch to setclasspath.sh that fixes the bug. |
Description
mschulz
2006-04-04 01:43:54 UTC
Created attachment 18029 [details]
patch for setclasspath.bat to make it consistent with the .sh version
Here is a patch to setclasspath.bat to make it consistent with Remy's changes
he made to setclasspath.sh some 14 months ago.
I'm hesitant to always include tools.jar now that we don't strictly need a JDK, and I'm not sure what the unix version does it. Consistency is a good goal but in this case I think we may need to go the other way, i.e. remove tools.jar from one script instead of adding it to the other. Please bring this up on the tomcat dev mailing list for discussion. Created attachment 18120 [details]
Patch to setclasspath.sh that fixes the bug.
This patch fixes the bug in my 5.5.16 installation such that I can then start
Tomcat with:
$CATALINA_HOME/bin/catalina.sh javac start
and it will use the Ant Java compiler to compile JSPs without copying
tools.jar anywhere.
I believe this bug affects the latest source, and this patch fixes it, at
least on the non-windows OSs.
This has been fixed in trunk along with a handful of other script niggles. All the fixes have been proposed for 5.5.x and 6.0.x The fixes for the niggles have been applied and will be in 5.5.28 and 6.0.21. The main fix was veto'd due to the incomplete implementation of the javac option and lack of documentation about what it was supposed to do. A new patch has been proposed which removes the javac option. tools.jar can be added to the classpath using the setenv script. This has been fixed in 6.0.x and 5.5.x and will be in 6.0.21 and 5.5.28 onwards. |