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 explicit module info #792

Merged
merged 1 commit into from
Apr 30, 2024
Merged

Add explicit module info #792

merged 1 commit into from
Apr 30, 2024

Conversation

bowbahdoe
Copy link
Contributor

Adds an explicit module-info to the final artifact

@belaban belaban merged commit fba6747 into belaban:master Apr 30, 2024
@belaban
Copy link
Owner

belaban commented May 1, 2024

Hmm... compilation with ant and mavcen succeeds, but IntelliJ complains:
java: module not found: org.apache.logging.log4j.core
java: module not found: org.apache.logging.log4j
java: module not found: org.slf4j

@bowbahdoe
Copy link
Contributor Author

This is perhaps not helpful, but IntelliJ is working fine on my machine. Maybe you need to upgrade your IJ?

@belaban
Copy link
Owner

belaban commented May 1, 2024

Still the same problem, I'm on 2023.2.6

@bowbahdoe
Copy link
Contributor Author

Thats the same version as me.

I nuked my local and re-cloned from github and all was fine. Maybe its an issue as to whether its a "maven project" in intellij's eyes? I'm also on a mac if that ends up mattering.

(I definitely have incentive to figure out whats going on - partially so you don't revert and partially so I can be aware of this in the future)

@tristantarrant
Copy link
Collaborator

tristantarrant commented May 1, 2024 via email

@belaban
Copy link
Owner

belaban commented May 2, 2024

This is driving me crazy and now I have a new error: Exception in thread "main" java.lang.NoClassDefFoundError: production/JGroups/module-info is not a class because access_flag ACC_MODULE is set.

Same issue regardless whether I import the project as maven or from srcs (ant).

I'm removing module-info.java, as it prevents me from working. I'll leave the changes in build.xml and pom.xml.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
3 participants