#426 module-info.java is placed in the wrong jar

viktor44 commented 1 year ago

Description

Fixes #426

File module-info.class was moved from META-INF/versions/9 to the root folder of the jar.

Types of changes

[x] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)

Checklist:

[ ] My change requires a change to the documentation.
[ ] I have updated the documentation accordingly.
[ ] I have added tests to cover my changes.

MALPI commented 1 year ago

I will take a look in the next days.

viktor44 commented 11 months ago

Up! 😄

otrosien commented 3 months ago

@viktor44 hey there, sorry for the long silence. The PR looks good, but I do have some questions.

mvn install -Dmaven.test.skip=true

Running this locally, adds the module-info.class to META-INF/versions/9/module-info.class. Maybe we somehow missed publishing this properly in the previous releases. Reading moditect README, setting jvmVersion to 9 or higher increases the compatiblity with old jar-scanning tools (which by now is an obsolete argument, I guess)

Secondly, let's upgrade the maven plugin to the current stable version.

And finally, please sign-off your commit, using git commit --amend --signoff.

zalando / problem