Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jigsaw support #117

Closed
wants to merge 7 commits into from
Closed

Jigsaw support #117

wants to merge 7 commits into from

Conversation

lpandzic
Copy link

@lpandzic lpandzic commented Feb 27, 2017

@cowtowncoder
Copy link
Member

Would it be possible to add bit of description of what changes are, and for why?

@lpandzic lpandzic changed the title #1536 Jigsaw support Mar 5, 2017
@lpandzic
Copy link
Author

lpandzic commented Mar 5, 2017

Sure.


<osgi.export>com.fasterxml.jackson.annotation.*;version=${project.version}</osgi.export>
<maven-compiler-plugin.version>3.6.1</maven-compiler-plugin.version>
<maven-bundle-plugin.version>3.2.0</maven-bundle-plugin.version>
Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

version upgrades are required for both in order to handle maven-module.java

<instructions>
<_noee>true</_noee>
<_failok>true</_failok>
<Include-Resource>module-info.class=target/classes/module-info.class</Include-Resource>
Copy link
Author

@lpandzic lpandzic Mar 5, 2017

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

required for bundle plugin to include module-info.class in built jar (and in turn, to build modular jar)

<configuration>
<instructions>
<_noee>true</_noee>
<_failok>true</_failok>
Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

without failok build fails:
[ERROR] Bundle com.fasterxml.jackson.core:jackson-annotations:bundle:2.9.0-SNAPSHOT : Invalid class file module-info.class (java.lang.ArrayIndexOutOfBoundsException: 19)

<version>${maven-bundle-plugin.version}</version>
<configuration>
<instructions>
<_noee>true</_noee>
Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

without noee build fails:
[ERROR] An internal error occurred
java.lang.IllegalArgumentException: Null query

@cowtowncoder
Copy link
Member

Ok so... does this mean that only JDK9 build would be enabled for Travis? But also that that version is not yet available?

@cowtowncoder
Copy link
Member

Before JDK 9, what we can do is #123 which I can add for 2.9.1 (unusual change for patch version, but makes sense here).

@cowtowncoder
Copy link
Member

Closing in favor of new attempt for 2.10

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.

None yet

2 participants