Maven 1.x JNLP Plugin

'Update Manifest' feature should be optional

Details

  • Type: Improvement Improvement
  • Status: Open Open
  • Priority: Major Major
  • Resolution: Unresolved
  • Affects Version/s: None
  • Fix Version/s: 1.4.1
  • Component/s: None
  • Labels:
    None
  • Environment:
    Mac OS X, Java 1.4.2
  • Number of attachments :
    1

Description

The maven-jnlp-plugin strips out all JAR manifest attributes and replaces them with a bare-bones manifest that looks like this:

Created-By: Apache Maven
Manifest-Version: 1.0
(Plus the SHA1-Digest attributes for each .class entry)

Apparently this was to work around bugs in Java Web Start in the 1.3.x versions of Java. My App requires 1.4.x, so I created a patched version of maven-jnlp-plugin that adds a property called 'maven.jnlp.update.manifest' which defaults to 'true' for backward compatibility.

Setting maven.jnlp.update.manifest=false will preserve attributes in the signed JNLP JARs and seems to work fine with JWS in Java 1.4.2.

I'm hoping someone will commit the patch and make a 1.4.2 release of the plugin...

Activity

Hide
M. Sean Gilligan added a comment -

This is the patch that I've tested in three scenarios: 1) Using default true value of the maven.jnlp.update.manifest property, 2) Overriding with false in project.properties, 3) overriding with true in project.properties. They all work correctly on OS X 10.3, with Java 1.4.2.

Show
M. Sean Gilligan added a comment - This is the patch that I've tested in three scenarios: 1) Using default true value of the maven.jnlp.update.manifest property, 2) Overriding with false in project.properties, 3) overriding with true in project.properties. They all work correctly on OS X 10.3, with Java 1.4.2.
Hide
Melloware added a comment -

I also have this issue where I need the manifest.mf to not be modified. This fix is perfect.

Show
Melloware added a comment - I also have this issue where I need the manifest.mf to not be modified. This fix is perfect.

People

Vote (1)
Watch (2)

Dates

  • Created:
    Updated: