-
Notifications
You must be signed in to change notification settings - Fork 1.1k
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
Can't find micronautserviceconfigs when running as jlink-ed application #10982
Conversation
You can write the test in JUnit 5 / java if it is easier for you |
could you sign the CLA? Thanks. |
@@ -187,6 +189,20 @@ private void findMicronautMetaServiceConfigs(BiConsumer<URI, String> consumer) t | |||
uniqueURIs.add(uri); | |||
} | |||
|
|||
if(uniqueURIs.isEmpty()) { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Not 100% sure if this emtpy-check is correct. For JLink it is, which was the usecase. In case of combining classpath and modulepath this might need to be revisited, although I wonder if people would do that.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Is test.jar
missing from the commit? The test appears to be failing and "build/resources/test/test.jar"
doesn't exist
//cc @rfscholte |
that's absolutely possible. Currently enjoying JCrete, while this code is still at home. I'll come back on this next week. |
thank you! |
unfortunately seems to break native image support. Let me investigate what can be done |
Thanks for the contribution! |
These few lines of code should be the base of the fix. And it actually doesn't require a predefined set. Tests are still missing as I'm struggling with the spock support in my IDE.
This fixes #10842