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

Add JDK9 Module name #32

Open
benoitx opened this issue Dec 15, 2017 · 6 comments
Open

Add JDK9 Module name #32

benoitx opened this issue Dec 15, 2017 · 6 comments
Assignees

Comments

@benoitx
Copy link
Contributor

benoitx commented Dec 15, 2017

Using method described in
https://dzone.com/articles/automatic-module-name-calling-all-java-library-maintainers

benoitx added a commit that referenced this issue Dec 15, 2017
@benoitx benoitx self-assigned this Dec 15, 2017
@martindiphoorn
Copy link
Collaborator

Why is this stil open?

@benoitx
Copy link
Contributor Author

benoitx commented Nov 22, 2018 via email

@martindiphoorn
Copy link
Collaborator

We should probably also add a module-info.java

See below site for a tutorial:
https://blog.codefx.org/java/java-module-system-tutorial/

@benoitx
Copy link
Contributor Author

benoitx commented Dec 5, 2018 via email

@martindiphoorn
Copy link
Collaborator

I did have a quicklook, but if i try to compile with java 11 i receive an error with osgi. And i don't have the time to look into that.

java.lang.NullPointerException
        at aQute.bnd.header.OSGiHeader.quote(OSGiHeader.java:150)

Also i don't know if i make it a java 11 module it will still work in osgi.

I think it would also be a good thing to cleanup our pom's a little bit. Make sure that the parent contains all the versions in properties. And the modules are using those. Now java version is defined in more then one place.

An other thing i don't know is this parent:

    <parent>
        <groupId>org.sonatype.oss</groupId>
        <artifactId>oss-parent</artifactId>
        <version>7</version>
    </parent>

And if it will be a problem with java 11.

@benoitx
Copy link
Contributor Author

benoitx commented Dec 6, 2018 via email

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

No branches or pull requests

2 participants