[icedtea-web] RFC: PR618 - Can't install OpenDJ, JavaWebStart fails with Input stream is null error.

Dr Andrew John Hughes ahughes at redhat.com
Mon Sep 26 17:38:44 PDT 2011


On 11:44 Mon 19 Sep     , Omair Majid wrote:
> Hi,
> 
> The attached patch fixes PR618. There is an old thread [1] that traces 
> the evolution of this patch.
> 
> The patch tires to make sure that resources marked as 'lazy' are 
> searched too if a resource is not found.
> 
> ChangeLog
> 2011-09-19  Omair Majid  <omajid at redhat.com>
> 
>    PR618
>    * NEWS: Update.
>    * netx/net/sourceforge/jnlp/runtime/JNLPClassLoader.java
>    (getResource): Rename to ...
>    (findResource): New method.
>    (findResources): If resource can not be found, search in lazy
>    resources.
>    (findResourcesBySearching): New method.
> 
> Any thoughts or comments?
> 

Sorry, I thought I replied to this.

My only comment was as to the addition of @Override; I don't think this
is necessary and it can cause issues with older compilers (there's an
issue with this, IcedTea6 1.8 and the last security patch for instance).

Otherwise, it's good to go.

> Cheers,
> Omair
> 
> [1] 
> http://thread.gmane.org/gmane.comp.java.openjdk.distro-packaging.devel/11869

-- 
Andrew :)

Free Java Software Engineer
Red Hat, Inc. (http://www.redhat.com)

Support Free Java!
Contribute to GNU Classpath and IcedTea
http://www.gnu.org/software/classpath
http://icedtea.classpath.org
PGP Key: F5862A37 (https://keys.indymedia.org/)
Fingerprint = EA30 D855 D50F 90CD F54D  0698 0713 C3ED F586 2A37



More information about the distro-pkg-dev mailing list