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

Issue with the new mutiny module-info.java (1.3.1) #835

Closed
jponge opened this issue Jan 30, 2022 Discussed in #833 · 1 comment · Fixed by #836
Closed

Issue with the new mutiny module-info.java (1.3.1) #835

jponge opened this issue Jan 30, 2022 Discussed in #833 · 1 comment · Fixed by #836
Labels
bug Something isn't working
Milestone

Comments

@jponge
Copy link
Member

jponge commented Jan 30, 2022

Discussed in #833

Originally posted by ljtfreitas January 29, 2022
Hi all 👋 ,

I'm working in a library that depends from mutiny. I'm using 1.2.0 version, and my module-info is:

module my.module {

    requires io.smallrye.mutiny;
}

And all it's working as expected. So, I'm trying to update to 1.3.1 version from mutiny, and now my compilation is failing because, since 1.3.0 version, mutiny does not export packages anymore; instead, it's declare the module with the open directive.

As far as I know about Java modules (not so much, I need to say :) ), a open module indicate that public types of the module are accessible, but just in runtime, not for compile time.

So, how can I fix this in order to upgrade the version? 🤔

Thank you so much, people.

@jponge jponge added the bug Something isn't working label Jan 30, 2022
@jponge jponge added this to the 1.4.0 milestone Jan 30, 2022
@jponge
Copy link
Member Author

jponge commented Jan 30, 2022

Confirmed, an open module is open for deep reflection, but this does not mean that all packages get exported.

@jponge jponge linked a pull request Jan 30, 2022 that will close this issue
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant