-
Notifications
You must be signed in to change notification settings - Fork 101
JPMS #465
Comments
PRs welcome |
Are you trying to maintain support for pre java 9? |
I'm not sure of the Java landscape these days. What does Android compatibility entail? |
I don't know what android's up to. Automatic module naming is backwards compatible for sure though. |
I believe recent-er versions of gradle can generate the automatic module manifest (I need to do some research) are you amenable to upgrading if that's the path of least resistance? |
Absolutely. |
Honestly, I went to do this, this morning, but I can't get the project to build. I'm not sure why you've done some things... so I don't want to mess around with a project that doesn't build
to fix this you might want to upgrade to gradle 8 (it was introduced in 7, but I'm not sure which version), and use the Java Toolchain feature. I'm only guessing at what you're trying to do though. to fix the actual issue I brought up here it should be as simple as jar {
manifest {
attributes 'Automatic-Module-Name': 'org.inferred.freebuilder'
}
} https://stackoverflow.com/a/53912585/206466 https://blog.joda.org/2017/05/java-se-9-jpms-automatic-modules.html |
hacky workaround (mostly using it to decide if freebuilder works for me, if not... then I have to decide what I want to do about that) plugins {
id("org.gradlex.extra-java-module-info").version("1.+")
}
extraJavaModuleInfo {
automaticModule("org.inferred:freebuilder", "org.inferred.freebuilder")
failOnMissingModuleInfo.set(false)
} not sure if if this adds the manifest to the jar, simply gives it a name on the cli or what https://github.com/gradlex-org/extra-java-module-info |
That error's coming from these lines here: FreeBuilder/gradle/java-compatibility.gradle Lines 34 to 35 in 3a38f58
It's supposed to write out an error message, but apparently the |
Yeah, I'm just thinking there might be a better way to do whatever you're trying to do than there was in 6. You should just be able to copy and paste those jar lines somewhere though and have add the manifest.MF when you build the jar. I haven't tested it though because... Oh because I didn't want to spend a ton of time making your build work. Especially with My IDE. |
please add a module-info or automatic module name so this can be used with java modules
The text was updated successfully, but these errors were encountered: