Skip to content

Fix Java 9 + regression, and layer in mavenization#19

Closed
ProgrammerDan wants to merge 3 commits intoEnderWiggin:masterfrom
ProgrammerDan:maven
Closed

Fix Java 9 + regression, and layer in mavenization#19
ProgrammerDan wants to merge 3 commits intoEnderWiggin:masterfrom
ProgrammerDan:maven

Conversation

@ProgrammerDan
Copy link
Copy Markdown

Let me know if you'd prefer just the Java 9 Buffer fix, but basically if you compile using Java 9 or later (I am compiling using Java 11 locally) the application fails due to a change in Java 9 involving the method return types of various ByteBuffer implementations. See apache/felix#114 for details. This impacts a handful of methods and files, which I've updated as a cherry-pickable commit.

As well, I don't prefer working with Ant by itself, so I've mavenized the project as well. Produces equivalent output to the target folder instead of the build folder, does all equivalent work as the Ant build.xml but via maven. Invoke the parent pom via mvn clean package -f parent-pom.xml . Any non-mavenized resources jars (like loftar's library, and javax.jnlp) are replicated in a local "maven repo" for resolution.

I don't know if you're open to outside contribution, or if this represents anything of value for you, but figured I'd attempt the contribute regardless. Have a great day.

@wafflekat
Copy link
Copy Markdown

Hey that's pretty dope

@EnderWiggin
Copy link
Copy Markdown
Owner

It is probably better to submit clear PR with fix only to loftar.

@ProgrammerDan
Copy link
Copy Markdown
Author

ProgrammerDan commented Apr 3, 2021 via email

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.

3 participants