steamworks4j icon indicating copy to clipboard operation
steamworks4j copied to clipboard
verified publisher icon code-disaster

jigsaw/module-info.java support?

Open chengenzhao opened this issue 5 years ago • 5 comments

Hi,

It would be nice to have a modular info for this project since we are using Java version 9+ to build the game many dependencies we are using have been modularized except this one

Screen Shot 2021-01-20 at 8 31 24 PM

thanks

chengenzhao avatar Jan 20 '21 12:01 chengenzhao

For my own opinion, I HATE jigsaw. But adding module-info.java seems no harm(to no-jigsaw users).

XenoAmess avatar Feb 09 '21 11:02 XenoAmess

I second this, it would be great to get module support. If necessary I can help get a PR together, or help answer any questions you have on implementing it.

tlf30 avatar Oct 11 '22 01:10 tlf30

@code-disaster I am looking at putting together a PR. I have hit a major blocking issue in making a simple migration using automation module names. Shared package between steamworks4j and steamworks4j-server jars. This is an issue because JMS does not allow classes in the same package from two different modules. A solution would be to move everything in the steamworks4j-server jar into its own classpath, such as com.codedisaster.steamworks.server, but because of the use of package private (protected) access between the two jars, this is not possible. Perhaps the best solution would be to merge the two jars into a single jar.

Thoughts?

EDIT: Please see draft pr #124

tlf30 avatar Jan 10 '23 16:01 tlf30

Bump

bowbahdoe avatar Dec 25 '23 05:12 bowbahdoe