Re: [PATCH] Small fix for Jetty 4.2.14

[Mark Wielaard]

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 <address@hidden>
> 
>         * 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 <address@hidden>

       * 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