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 descriptor fails if one jar of many has issues #12

Closed
maxandersen opened this issue Feb 24, 2023 · 2 comments
Closed

module descriptor fails if one jar of many has issues #12

maxandersen opened this issue Feb 24, 2023 · 2 comments
Assignees
Labels
bug Something isn't working released Issue has been released

Comments

@maxandersen
Copy link

jarviz module descriptor --classpath=`jbang info classpath env@jbangdev`
java.lang.module.FindException: Unable to derive module descriptor for /Users/manderse/.m2/repository/de/vandermeer/char-translation/0.0.2/char-translation-0.0.2.jar
Unable to derive module descriptor for /Users/manderse/.m2/repository/de/vandermeer/char-translation/0.0.2/char-translation-0.0.2.jar

I would have expected it would list char-translation-0.0.2.jar to not have module descriptor together with rest of the classpath entries.

@maxandersen maxandersen added the bug Something isn't working label Feb 24, 2023
@aalmiray
Copy link
Contributor

The current rule is to let the JVM derive the automatic module name is it would normally do. I suppose the exception could be caught when running in batch mode, after all reports already have an entry (named reason) for when such errors may occur.

@aalmiray
Copy link
Contributor

aalmiray commented Mar 5, 2023

Released in v0.3.0 -> https://github.com/kordamp/jarviz/releases/tag/v0.3.0

@aalmiray aalmiray added the released Issue has been released label Mar 5, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working released Issue has been released
Projects
None yet
Development

No branches or pull requests

2 participants