You may want this one, also. Turns out that jasper.sh is mildly broken on HEAD. It doesn't grab the libraries in shared/lib or the classes in shared/classes.
I'm guessing that JspC isn't the most heavily exercised part of the system.
<g>
Index: jasper.sh
===================================================================
RCS file: /home/cvspublic/jakarta-tomcat-4.0/jasper/src/bin/jasper.sh,v
retrieving revision 1.5
diff -u -r1.5 jasper.sh
--- jasper.sh 2001/09/13 15:22:45 1.5
+++ jasper.sh 2001/12/19 00:10:39
@@ -35,7 +35,7 @@
done
JASPER_HOME_1=`dirname "$PRG"`/..
- echo "Guessing JASPER_HOME from catalina.sh to ${JASPER_HOME_1}"
+ echo "Guessing JASPER_HOME from jasper.sh to ${JASPER_HOME_1}"
if [ -d ${JASPER_HOME_1}/conf ] ; then
JASPER_HOME=${JASPER_HOME_1}
echo "Setting JASPER_HOME to $JASPER_HOME"
@@ -80,6 +80,10 @@
done
CP=$CP:"$JASPER_HOME/common/classes"
for i in $JASPER_HOME/common/lib/*.jar ; do
+ CP=$CP:"$i"
+done
+CP=$CP:"$JASPER_HOME/shared/classes"
+for i in $JASPER_HOME/shared/lib/*.jar ; do
CP=$CP:"$i"
done
> -----Original Message-----
> From: Kin-Man Chung [mailto:[EMAIL PROTECTED]]
> Sent: Tuesday, December 18, 2001 5:12 PM
> To: [EMAIL PROTECTED]
> Subject: RE: [PATCH] Bug 5471 - JspC broken when compiling webapps
>
>
> I responsed in haste, without reading your patch carefully. :( My
> apologies.
>
> I'll get the patch in, later. Thanks.
>
>
> > Date: Tue, 18 Dec 2001 16:44:42 -0500
> > From: Steve Downey <[EMAIL PROTECTED]>
> > Subject: RE: [PATCH] Bug 5471 - JspC broken when compiling webapps
> > To: 'Tomcat Developers List' <[EMAIL PROTECTED]>
> > MIME-version: 1.0
> > Delivered-to: mailing list [EMAIL PROTECTED]
> > Mailing-List: contact [EMAIL PROTECTED];
> run by ezmlm
> > X-Spam-Rating: daedalus.apache.org 1.6.2 0/1000/N
> > List-Post: <mailto:[EMAIL PROTECTED]>
> > List-Subscribe: <mailto:[EMAIL PROTECTED]>
> > List-Unsubscribe: <mailto:[EMAIL PROTECTED]>
> > List-Help: <mailto:[EMAIL PROTECTED]>
> > List-Id: "Tomcat Developers List" <tomcat-dev.jakarta.apache.org>
> >
> > The patch doesn't mangle the class file name, just the
> package name, so that
> > the resulting java file is put in a directory that matches
> the path of the
> > jsp file. Just putting the java file in a subdirectory is
> insufficient,
> > though. Unless the packages are different, there isn't a
> way to disambiguate
> > the servlets that result from two index.jsp's in the web.xml file.
> >
> > Example:
> > input
> > /index.jsp
> > /subdir/index.jsp
> >
> > result
> > /index.java - contains class index
> > /subdir/index.java - contains class subdir.index
> >
> > If the -p option is used, eg -p com.netfolio.jspservlets
> > /com/netfolio/jspservlets/index.java - contains class
> > com.netfolio.jspservlets.index
> > /com/netfolio/jspservlets//subdir/index.java - contains class
> > com.netfolio.jspservlets.subdir.index
> >
> >
> >
> > > -----Original Message-----
> > > From: Kin-Man Chung [mailto:[EMAIL PROTECTED]]
> > > Sent: Tuesday, December 18, 2001 4:37 PM
> > > To: [EMAIL PROTECTED]
> > > Subject: Re: [PATCH] Bug 5471 - JspC broken when compiling webapps
> > >
> > >
> > > Believe or not, Jasper once mangled the file names in a way
> > > similar to what
> > > is in the patch. It was modified in response to a bug
> > > (Bugzilla is currently
> > > unavailable, so I can't look up the bug number). The filer
> > > complained that
> > > for a file with deeply nested path, the resultant file name
> > > is too long to
> > > work in Window, since there is a 250(?) character file name
> > > limit there.
> > >
> > > I think a better fix is not to mangle the path into the
> file name, but
> > > to put the .java (and .class) files in a directory structure
> > > that mirrors
> > > the .jsp structure. This is how it work currently, for the
> > > non -webapps
> > > case.
> > >
> > >
> > > > Date: Tue, 18 Dec 2001 15:47:25 -0500
> > > > From: Steve Downey <[EMAIL PROTECTED]>
> > > > Subject: [PATCH] Bug 5471 - JspC broken when compiling webapps
> > > > To: "[EMAIL PROTECTED] (E-mail)"
> > > <[EMAIL PROTECTED]>
> > > > MIME-version: 1.0
> > > > Delivered-to: mailing list [EMAIL PROTECTED]
> > > > Mailing-List: contact [EMAIL PROTECTED];
> > > run by ezmlm
> > > > X-Spam-Rating: daedalus.apache.org 1.6.2 0/1000/N
> > > > List-Post: <mailto:[EMAIL PROTECTED]>
> > > > List-Subscribe: <mailto:[EMAIL PROTECTED]>
> > > > List-Unsubscribe:
> <mailto:[EMAIL PROTECTED]>
> > > > List-Help: <mailto:[EMAIL PROTECTED]>
> > > > List-Id: "Tomcat Developers List"
> <tomcat-dev.jakarta.apache.org>
> > > >
> > > > This patch changes
> > > CommandLineContext.getServletPackageName() to return a
> > > > package name based on the path to the JSP as well as the
> > > package name
> > > > supplied on the command line.
> > > >
> > > > Without a change like this, a webapp that has files with
> > > the same name, such
> > > > as index.jsp, in more than one place is compiled
> > > incorrectly, with both JSP
> > > > files being mapped to the same servlet.
> > > >
> > > > With this patch, the java files are placed in the correct
> > > directories, and
> > > > generation of web.xml works.
> > > >
> > > >
> > > > <><><><><><><><><><><><><><><><><><><><><>This electronic
> > > mail transmission
> > > > may contain confidential information and is intended only
> > > for the person(s)
> > > > named. Any use, copying or disclosure by any other person
> > > is strictly
> > > > prohibited. If you have received this transmission in
> > > error, please notify
> > > > the sender via e-mail.
> <><><><><><><><><><><><><><><><><><><><><>
> > > >
> > > >
> > >
> > >
> > > --
> > > To unsubscribe, e-mail:
> > > <mailto:[EMAIL PROTECTED]>
> > > For additional commands, e-mail:
> > > <mailto:[EMAIL PROTECTED]>
> > >
> > <><><><><><><><><><><><><><><><><><><><><>This electronic
> mail transmission
> > may contain confidential information and is intended only
> for the person(s)
> > named. Any use, copying or disclosure by any other person
> is strictly
> > prohibited. If you have received this transmission in
> error, please notify
> > the sender via e-mail. <><><><><><><><><><><><><><><><><><><><><>
> >
> > --
> > To unsubscribe, e-mail:
> <mailto:[EMAIL PROTECTED]>
> > For additional commands, e-mail:
> <mailto:[EMAIL PROTECTED]>
> >
>
>
> --
> To unsubscribe, e-mail:
> <mailto:[EMAIL PROTECTED]>
> For additional commands, e-mail:
> <mailto:[EMAIL PROTECTED]>
>
<><><><><><><><><><><><><><><><><><><><><>This electronic mail transmission
may contain confidential information and is intended only for the person(s)
named. Any use, copying or disclosure by any other person is strictly
prohibited. If you have received this transmission in error, please notify
the sender via e-mail. <><><><><><><><><><><><><><><><><><><><><>
jasper.sh.patch
Description: Binary data
-- To unsubscribe, e-mail: <mailto:[EMAIL PROTECTED]> For additional commands, e-mail: <mailto:[EMAIL PROTECTED]>
