java-vorbis-support icon indicating copy to clipboard operation
java-vorbis-support copied to clipboard

Problems with Java 9 and above

Open goxr3plus opened this issue 6 years ago • 8 comments

Hello Trillarion, Automatic - Module - Name has problems with the file Name for Java 9 and above, can you please do a new release with the name.

java-vorbis-support-x. x. x

Thank you very much :)

goxr3plus avatar May 01 '19 09:05 goxr3plus

I will have a look in the next couple of days.

Trilarion avatar May 02 '19 21:05 Trilarion

Well actually, renaming jar file is good to a certain degree from the viewpoint of following convention, but it would be far more helpful if you added an Automatic-Module-Name attribute to the JAR manifest. To do so you could add the following code into build.gradle:

jar {
    manifest {
        attributes 'Automatic-Module-Name': moduleName
    }
}

where moduleName should be replaced to certain module name for this project. Hope this will work fine but I'm not sure.

TonalidadeHidrica avatar May 03 '19 17:05 TonalidadeHidrica

I think I got it right with the recent commits. Please let me know if everything is working for you now.

Trilarion avatar Jun 03 '19 22:06 Trilarion

@TonalidadeHidrica Oou here you are ma bro :)

goxr3plus avatar Jun 04 '19 10:06 goxr3plus

@Trilarion Now it Java9 ready but support for Java 8 dropped .... lol . Thank you by the way .

goxr3plus avatar Jun 04 '19 10:06 goxr3plus

@goxr3plus Must have been carried away. Now Java 8 and 9 should both be fine.

Trilarion avatar Jun 05 '19 20:06 Trilarion

I created a fork that is fully jlink-ready. It seems jlink dislikes automatic modules greatly - all modules need to be explicitly named to function. Alas, I'm using JDK 13, so that's what I'm requiring, but I'm sure a small adjustment to the Gradle script for Java 9+ can get this working too. I'm not knowledgeable enough on Gradle to fix this within Gradle, so I made a Gradle-less solution that works just fine.

retropipes avatar Nov 30 '19 14:11 retropipes

If there is time I will try it out. Ideally I would have a solution that works with Java 8 as well as with newer Javas and with all modes of packaging.

Trilarion avatar Dec 03 '19 09:12 Trilarion