Skip to content

Main-Class in jna-platform.jar collides with java 9 module system#804

Merged
matthiasblaesing merged 1 commit intojava-native-access:masterfrom
matthiasblaesing:remove_platform_main_class
May 14, 2017
Merged

Main-Class in jna-platform.jar collides with java 9 module system#804
matthiasblaesing merged 1 commit intojava-native-access:masterfrom
matthiasblaesing:remove_platform_main_class

Conversation

@matthiasblaesing
Copy link
Copy Markdown
Member

The java 9 module system expects the main class to reside in the same
jar, as the MANIFEST.MF. For the jna.jar the use case is valid to
retrieve the current version of JNA, as jna.jar must be available
to run jna-platform.jar, there is no use-case to have the Main-Class
defined.

As reference the explanation from the jigsaw-dev mailing list:

http://mail.openjdk.java.net/pipermail/jigsaw-dev/2017-April/012151.html

The java 9 module system expects the main class to reside in the same
jar, as the MANIFEST.MF. For the jna.jar the use case is valid to 
retrieve the current version of JNA, as jna.jar must be available
to run jna-platform.jar, there is no use-case to have the Main-Class
defined.

As reference the explanation from the jigsaw-dev mailing list:

http://mail.openjdk.java.net/pipermail/jigsaw-dev/2017-April/012151.html
@matthiasblaesing matthiasblaesing force-pushed the remove_platform_main_class branch from 64b08db to f942fc0 Compare May 14, 2017 18:18
@matthiasblaesing matthiasblaesing merged commit 4450862 into java-native-access:master May 14, 2017
@matthiasblaesing matthiasblaesing deleted the remove_platform_main_class branch August 12, 2017 13:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant