Hi, On Thu, 2003-11-13 at 01:29, Dalibor Topic wrote: > Hi, > > attached is a small fix for Jetty 4.2.14 that is needed to get Jetty to > work with Classpath's file URL Connection. Jetty otherwise breaks on > startup and complains that webapps/templates can not be found, despite > that it exists. Jetty wants to access the path of the file permission on > a file URL connection to determine the existence of a file, apparently. > > The fix overwrites getPermission the file URL connection to return a > FilePermission with the file path and a read permission.
Thanks. I also wrote a Mauve test for this. (The test still fails because our file Handler.parseURL() seem broken. But if that is fixed, your patch makes the new Mauve test succeed.) 2003-11-13 Dalibor Topic <[EMAIL PROTECTED]> > > * libraries/javalib/gnu/java/net/protocol/file/Connection.java: > Added imports for java.io.FilePermission and > java.security.Permission. > (permission) New field. > (DEFAULT_PERMISSION) New constant. > (Connection) Create a FilePermission with permission to read > from file. > (getPermission) Overwrite getPermission to return a > FilePermission. > Applied as follows: 2003-11-13 Dalibor Topic <[EMAIL PROTECTED]> * gnu/java/net/protocol/file/Connection.java (permission): New field. (DEFAULT_PERMISSION): New constant. (Connection): Create a FilePermission with permission to read file. (getPermission): Overwrite getPermission to return a FilePermission. You don't need to mention really trivial changes such as new imports. Note the placement of the ':' between what was changed and the description of the change. Maybe kaffe could in the long run switch to a separate ChangeLog file in the libraries/javalib directory so that we can even share the complete ChangeLog messages between the projects as we already do with gcj libjava. Cheers, Mark
signature.asc
Description: This is a digitally signed message part
_______________________________________________ Classpath mailing list [EMAIL PROTECTED] http://mail.gnu.org/mailman/listinfo/classpath