tomcat-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Larry Isaacs <Larry.Isa...@sas.com>
Subject RE: svn commit: r810132 - /tomcat/tc6.0.x/trunk/STATUS.txt
Date Fri, 04 Sep 2009 13:22:58 GMT
On Windows prepending "file://" to "C:\somedir\somefile" results in "file://C:/somedir/somefile".
 I'm pretty sure URL will think "C" is the host name and fail if you try to use a URL object
to read the file.  I think it was the same issue on Linux when prepending "file:/" to "/somedir/somefile",
i.e. "somedir" was seen as a host name.  I think you need to ensure there is one slash or
three to avoid inadvertently specifying a host name.

Larry

> -----Original Message-----
> From: Filip Hanik - Dev Lists [mailto:devlists@hanik.com]
> Sent: Thursday, September 03, 2009 5:52 PM
> To: Tomcat Developers List
> Subject: Re: svn commit: r810132 - /tomcat/tc6.0.x/trunk/STATUS.txt
>
> The URL has to start with "file://"
>
> So the VirtualWebappLoader, can take a regular file path, and should
> simply prepend file://, not file: or file:/
>
> Filip
>
> On 09/01/2009 12:48 PM, Larry Isaacs wrote:
> > Hi Filip,
> >
> > I have a vague recollection about tripping over this issue while
> working on the Tomcat support in the Eclipse Web Tools Platform.  WTP
> has its own version(s) of VirtualWebappLoader, called WtpWebappLoader,
> to address it.  If I'm recalling the issue correctly, pre-pending
> "file:/" worked on Windows, but not on Linux since an absolute path
> ended up as "file://somedir/...".  I believe with the change below,
> pre-pending "file://" will work for Linux, but not for Windows, i.e.
> "file://C:/somedir/..." is going to have problems.  If it helps, I went
> with the URL form "file:/somedir/..." and used the following code
> snippet to get it working for Windows and Linux in WtpWebappLoader:
> >
> >
> >              String path = file.getAbsolutePath();
> >              if (path.startsWith("/")) {
> >                  path = "file:" + path;
> >              } else {
> >                  path = "file:/" + path;
> >              }
> >              if (file.isDirectory()) {
> >                  addRepository(path + "/");
> >              } else {
> >                  addRepository(path);
> >              }
> >
> > Cheers,
> > Larry
> >
> >
> >
> >> -----Original Message-----
> >> From: fhanik@apache.org [mailto:fhanik@apache.org]
> >> Sent: Tuesday, September 01, 2009 1:22 PM
> >> To: dev@tomcat.apache.org
> >> Subject: svn commit: r810132 - /tomcat/tc6.0.x/trunk/STATUS.txt
> >>
> >> Author: fhanik
> >> Date: Tue Sep  1 17:21:59 2009
> >> New Revision: 810132
> >>
> >> URL: http://svn.apache.org/viewvc?rev=810132&view=rev
> >> Log:
> >> proposal
> >>
> >> Modified:
> >>      tomcat/tc6.0.x/trunk/STATUS.txt
> >>
> >> Modified: tomcat/tc6.0.x/trunk/STATUS.txt
> >> URL:
> >>
> http://svn.apache.org/viewvc/tomcat/tc6.0.x/trunk/STATUS.txt?rev=810132
> >> &r1=810131&r2=810132&view=diff
> >>
> =======================================================================
> >> =======
> >> --- tomcat/tc6.0.x/trunk/STATUS.txt (original)
> >> +++ tomcat/tc6.0.x/trunk/STATUS.txt Tue Sep  1 17:21:59 2009
> >> @@ -321,3 +321,25 @@
> >>     +1: funkman
> >>     -1:
> >>
> >> +
> >> +* Fix VirtualWebappClassLoader file directives
> >> +  Index: java/org/apache/catalina/loader/VirtualWebappLoader.java
> >> +===================================================================
> >> +--- java/org/apache/catalina/loader/VirtualWebappLoader.java
> >>        (revision 810099)
> >> ++++ java/org/apache/catalina/loader/VirtualWebappLoader.java
> >>        (working copy)
> >> +@@ -92,9 +92,9 @@
> >> +                 continue;
> >> +             }
> >> +             if (file.isDirectory()) {
> >> +-                addRepository("file:/" + file.getAbsolutePath() +
> >> "/");
> >> ++                addRepository("file://" + file.getAbsolutePath() +
> >> "/");
> >> +             } else {
> >> +-                addRepository("file:/" + file.getAbsolutePath());
> >> ++                addRepository("file://" + file.getAbsolutePath());
> >> +             }
> >> +         }
> >> +
> >> +  +1: fhanik
> >> +  -1:
> >> +
> >> +
> >>
> >>
> >>
> >> --------------------------------------------------------------------
> -
> >> To unsubscribe, e-mail: dev-unsubscribe@tomcat.apache.org
> >> For additional commands, e-mail: dev-help@tomcat.apache.org
> >>
> >>
> >
>
>
> ---------------------------------------------------------------------
> To unsubscribe, e-mail: dev-unsubscribe@tomcat.apache.org
> For additional commands, e-mail: dev-help@tomcat.apache.org
>

Mime
View raw message