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

Added manifest entry to the eventbus-java jar file #702

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

plumstone
Copy link

@plumstone plumstone commented Jun 1, 2022

Solves the warning mentioned in #507

@greenrobot-team
Copy link
Collaborator

Thanks! We'll have a look at some point if that is enough (note: I remember something about package names not being allowed to overlap, need to check if the Android artifact ever is supposed to use modules, too).

@plumstone
Copy link
Author

The build failed due to an error in the jdk setup process. Maybe we should consider updating the jdk to temurin (successor of adopt) https://github.com/actions/setup-java#basic

@greenrobot-team
Copy link
Collaborator

@plumstone Thanks for the info! Fixed in master branch, e.g. rebase your change onto or merge the changes from master.

@plumstone plumstone force-pushed the 507-java-9-modular-support-request branch from 7d6bbe4 to 73f15d7 Compare December 8, 2022 23:57
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.

2 participants