RFC: Netx - Use version based download protocol for JNLP files

Omair Majid omajid at redhat.com
Fri Jul 10 08:51:07 PDT 2009


This patch makes Netx use the version based download protocol for JNLP 

Netx tries to use the version based download protocol whenever possible 
to download files. This wasn't being done for JNLP file that were 
specified as extensions by other JNLP files. This patch fixes the issue 
making JNLP applications like aerith[1] work. This is also required to 
run JavaFX applications.

While adding that support, I noticed that function arguments that 
involved Urls and versions were inconsistent: some used function(URL, 
Version, blah) while others used function(URL, blah, Version). I have 
tried to fix them to be function(URL, Version, blah) consistently.

2009-07-10  Omair Majid  <omajid at redhat.com>

   * rt/net/sourceforge/jnlp/JNLPFile.java
   (JNLPFile): Delegate to the Version-based constructor.
   (JNLPFile): New constructor.
   (JNLPFile): Modified to take an additional version argument used in
   downloading the JNLP file.
   (openURL): Take an additional version argument and use when
   downloading the URL.
   * rt/net/sourceforge/jnlp/Launcher.java
   (toFile): Use the new JNLPFile constructor.
   * rt/net/sourceforge/jnlp/cache/Resource.java
   (Resource): Rearrange argument order.
   (getResource): Likewise. Fix parameters to constructor.
   * rt/net/sourceforge/jnlp/cache/ResourceTracker.java
   (addResource): Fix arguments to Resource.getResource.
   * rt/net/sourceforge/jnlp/runtime/JNLPClassLoader.java
   (getInstance): Take additional version argument and useit when
   creating a JNLPFile.
   (initializeExtensions): Use the extension version when requesting a

Any comments?


[1] https://aerith.dev.java.net/
-------------- next part --------------
A non-text attachment was scrubbed...
Name: netx-versioned-jnlp.patch
Type: text/x-patch
Size: 7497 bytes
Desc: not available
Url : http://mail.openjdk.java.net/pipermail/distro-pkg-dev/attachments/20090710/8396ada0/netx-versioned-jnlp.patch 

More information about the distro-pkg-dev mailing list