Re: jar manifest main class can contain /

[Mark Wielaard <mark at klomp dot org>]

Hi,

On Mon, Mar 19, 2001 at 10:21:59AM +1200, Bryce McKinlay wrote:
> Mark Wielaard wrote:
> 
> >         * gnu/gcj/runtime/FirstThread.java (main): replace / with . in
> >         jarMainClassName
> 
> Since the Jar URLClassLoader should be handling this format already, maybe a
> better fix would be for FirstThread or whatever to be explicitly creating its
> own Jar URLClassLoader and passing that to Class.forName() in the "gij -jar"
> case? I think to implement the correct 1.2 semantics we need to have a
> seperate ClassLoader instance loading interpreted classes anyway - right now I
> think that everything gets loaded by the one, system classloader which doesn't
> seem right.
Yes you are right. That would also make sure that the Class-Path attribute
in the jar file manifest is actually used (it seems to be ignored now).
I still have a uncommitted URLClassLoader for Classpath that I really should
merge with the libgcj version.

> But as a quick workaround, this is fine to commit.

OK. Done.

Thanks,

Mark
-- 
Stuff to read:
    <http://www.toad.com/gnu/whatswrong.html>
  What's Wrong with Copy Protection, by John Gilmore