Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Module info for picocli`s Spring Boot Starter not included in the released jar #2335

Closed
obfischer opened this issue Aug 28, 2024 · 7 comments
Labels
theme: module An issue or change related to JPMS modules type: bug 🐛
Milestone

Comments

@obfischer
Copy link

The module info for picocli`s module info descriptor, which is present in the Git repository, is not included in the resulting released jar. I checked this for serveral versions.

Here is the output of the jar tool listing the content of the jar

09:11 $ jar -tvf /opt/mavencache/info/picocli/picocli-spring-boot-starter/4.7.6/picocli-spring-boot-starter-4.7.6.jar
     0 Tue May 07 21:23:52 CEST 2024 META-INF/
   258 Tue May 07 21:23:52 CEST 2024 META-INF/MANIFEST.MF
     0 Tue May 07 21:23:52 CEST 2024 picocli/
     0 Tue May 07 21:23:52 CEST 2024 picocli/spring/
     0 Tue May 07 21:23:52 CEST 2024 picocli/spring/boot/
     0 Tue May 07 21:23:52 CEST 2024 picocli/spring/boot/autoconfigure/
  1560 Tue May 07 21:23:52 CEST 2024 picocli/spring/boot/autoconfigure/PicocliAutoConfiguration.class
  2584 Tue May 07 21:23:52 CEST 2024 picocli/spring/PicocliSpringFactory.class
     0 Tue May 07 21:23:52 CEST 2024 META-INF/spring/
    60 Tue May 07 21:23:52 CEST 2024 META-INF/spring/org.springframework.boot.autoconfigure.AutoConfiguration.imports
   123 Tue May 07 21:23:52 CEST 2024 META-INF/spring.factories

Please include the descriptor in the release. Thank you for all the work you do!

Relates to #1608

@remkop remkop added this to the 4.7.7 milestone Aug 28, 2024
@remkop remkop added type: bug 🐛 theme: module An issue or change related to JPMS modules labels Aug 28, 2024
@remkop
Copy link
Owner

remkop commented Aug 28, 2024

It looks like the module-info class is missing from all 4.7.6 jars except the main picocli jar...

@obfischer
Copy link
Author

If you have a version of branch with a fix, I would be happy to test it. Currently I have some problems getting in working with jlink and my custom module infor descriptor.

@obfischer
Copy link
Author

The missing module declaration leads to some warnings during the build

[INFO] --- compiler:3.13.0:compile (default-compile) @ foobar ---
[INFO] Recompiling the module because of changed dependency.
[INFO] Compiling 4 source files with javac [debug parameters release 22 module-path] to target/classes
[WARNING] /Users/obf/code/cgm/his-cli.d/initial-structure/cli/foobar/src/main/java/module-info.java:[12,32] Modul nicht gefunden: picocli.spring.boot.starter
[WARNING] /Users/obf/code/cgm/his-cli.d/initial-structure/cli/foobar/src/main/java/module-info.java:[16,32] Modul nicht gefunden: picocli.spring.boot.starter
[WARNING] /Users/obf/code/cgm/his-cli.d/initial-structure/cli/foobar/src/main/java/module-info.java:[12,32] Modul nicht gefunden: picocli.spring.boot.starter
[WARNING] /Users/obf/code/cgm/his-cli.d/initial-structure/cli/foobar/src/main/java/module-info.java:[16,32] Modul nicht gefunden: picocli.spring.boot.starter
[WARNING] /Users/obf/code/cgm/his-cli.d/initial-structure/cli/foobar/src/main/java/module-info.java:[12,32] Modul nicht gefunden: picocli.spring.boot.starter
[WARNING] /Users/obf/code/cgm/his-cli.d/initial-structure/cli/foobar/src/main/java/module-info.java:[16,32] Modul nicht gefunden: picocli.spring.boot.starter

Just for your information

@remkop
Copy link
Owner

remkop commented Aug 31, 2024

Fixed in the main branch.

@remkop remkop closed this as completed Aug 31, 2024
@obfischer
Copy link
Author

Hi @remkop, I will test it today. Haven't had the time before

@remkop
Copy link
Owner

remkop commented Sep 11, 2024

@obfischer I appreciate you taking the time to help with this! 🙏

@obfischer
Copy link
Author

@remkop I added to each of my Maven modules a module-info.java and not it works like a charm. Looking forward to a release.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
theme: module An issue or change related to JPMS modules type: bug 🐛
Projects
None yet
Development

No branches or pull requests

3 participants
@obfischer @remkop and others