-
Notifications
You must be signed in to change notification settings - Fork 3k
Deprecate quarkus.native.resources.excludes
#50445
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
base: main
Are you sure you want to change the base?
Deprecate quarkus.native.resources.excludes
#50445
Conversation
.includePattern(".*/iio-plugin.*properties$") // Texts for e.g. exceptions strings | ||
.includePattern(".*/.*pf$") // Default colour profiles | ||
.includeGlobs("**/iio-plugin*.properties", // Texts for e.g. exceptions strings | ||
"**/*.pf") // Default colour profiles |
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.
@Karm please confirm that the new globs capture what you intended to capture with the patterns. Thanks
extensions/jaxb/deployment/src/main/java/io/quarkus/jaxb/deployment/JaxbProcessor.java
Show resolved
Hide resolved
This comment has been minimized.
This comment has been minimized.
🎊 PR Preview 31e401d has been successfully built and deployed to https://quarkus-pr-main-50445-preview.surge.sh/version/main/guides/
|
This comment has been minimized.
This comment has been minimized.
The new `reachability-metadata.json` file doesn't allow the exclusion of resources and uses globs for the inclusion of them. See: * oracle/graal#7487 * oracle/graal#9048 * oracle/graal#12255
7b02ea8
to
abb226e
Compare
nativeResourcePatterns.produce(builder().includePatterns( | ||
"META-INF/.*.kotlin_module$", | ||
nativeResourcePatterns.produce(builder().includeGlobs( | ||
"META-INF/**/*.kotlin_module", | ||
"META-INF/services/kotlin.reflect.*", | ||
".*.kotlin_builtins") | ||
"**/*.kotlin_builtins") |
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.
@geoand please confirm that the new globs capture what you intended to capture with the patterns. Thanks
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.
I don't remember for sure, but I'm pretty sure the pattern wasn't meant to capture the files in an arbitrary depth
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.
Hmmm, I know for sure that *.kotlin_builtins
didn't work, so it was meant to capture at least "some" depth. Furthermore, even if not meant to do so .*
does capture arbitrary depth. What do you suggest?
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.
If this works and attempts did it, let's go ahead with it
Status for workflow
|
Status for workflow
|
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.
I will let individual people confirm but this looks good to me (even if having exclusions made kinda sense to me).
The new
reachability-metadata.json
file doesn't allow the exclusion ofresources and uses globs for the inclusion of them.
See:
resource-config.json
oracle/graal#7487Related to #41016