ppalaga commented on a change in pull request #2813:
URL: https://github.com/apache/camel-quarkus/pull/2813#discussion_r655499832
##########
File path: docs/modules/ROOT/pages/migration-guide/2.0.0.adoc
##########
@@ -19,3 +19,8 @@ The empty `camel-quarkus-main` artifact was removed
completely in Camel Quarkus
As long as your application depends on any other Camel Quarkus extension, it
is enough to remove the `camel-quarkus-main` dependency.
This is because all Camel Quarkus extensions transitively depend on
`camel-quarkus-core` where the the original `camel-quarkus-main` functionality
is hosted now.
+
+== `quarkus.camel.native.resources.*` replaced by
`quarkus.native.resources.includes`
+
+The `quarkus.camel.native.resources.include-patterns` and
`quarkus.camel.native.resources.exclude-patterns` configuration properties were
removed in Camel Quarkus 2.0.0.
+Please use `quarkus.native.resources.includes` and
`quarkus.native.resources.includes` instead respectively.
Review comment:
+1, thanks!
##########
File path:
extensions-core/core/runtime/src/main/java/org/apache/camel/quarkus/core/CamelConfig.java
##########
@@ -212,25 +212,15 @@
public static class ResourcesConfig {
/**
- * A comma separated list of Ant-path style patterns to match resources
- * that should be <strong>excluded</strong> from the native
executable. By default,
- * resources not selected by quarkus itself are ignored. Then,
inclusion
- * of additional resources could be triggered with
- * <code>includePatterns</code>. When the inclusion patterns is too
- * large, eviction of previously selected resources could be triggered
- * with <code>excludePatterns</code>.
+ * Replaced by {@code quarkus.native.resources.excludes} in Camel
Quarkus 2.0.0.
+ * Using this property throws an exception at build time.
*/
@ConfigItem
public Optional<List<String>> excludePatterns;
/**
- * A comma separated list of Ant-path style patterns to match resources
- * that should be <strong>included</strong> in the native executable.
By default,
- * resources not selected by quarkus itself are ignored. Then,
inclusion
- * of additional resources could be triggered with
- * <code>includePatterns</code>. When the inclusion patterns is too
- * large, eviction of previously selected resources could be triggered
- * with <code>excludePatterns</code>.
+ * Replaced by {@code quarkus.native.resources.includes} in Camel
Quarkus 2.0.0.
+ * Using this property throws an exception at build time.
*/
@ConfigItem
public Optional<List<String>> includePatterns;
Review comment:
Yep, it is there in `docs/modules/ROOT/pages/migration-guide/2.0.0.adoc`
##########
File path:
extensions-core/core/deployment/src/main/java/org/apache/camel/quarkus/core/deployment/CamelNativeImageProcessor.java
##########
@@ -217,31 +217,13 @@ void embedSelectResourcesInNativeExecutable(CamelConfig
config, ApplicationArchi
BuildProducer<NativeImageResourceBuildItem> resources) {
final ResourcesConfig resourcesConfig = config.native_.resources;
- if (!resourcesConfig.includePatterns.isPresent()) {
- LOGGER.debug("Not scanning resources for native inclusion as
include-patterns is not set");
- return;
+ if (resourcesConfig.includePatterns.isPresent()) {
+ throw new IllegalStateException(
+ "quarkus.camel.native.resources.include-patterns
configuration property was removed in Camel Quarkus 2.0.0. Use
quarkus.native.resources.includes instead.");
Review comment:
Yes, done in docs/modules/ROOT/pages/migration-guide/2.0.0.adoc
##########
File path:
extensions-core/core/runtime/src/main/java/org/apache/camel/quarkus/core/CamelConfig.java
##########
@@ -212,25 +212,15 @@
public static class ResourcesConfig {
/**
- * A comma separated list of Ant-path style patterns to match resources
- * that should be <strong>excluded</strong> from the native
executable. By default,
- * resources not selected by quarkus itself are ignored. Then,
inclusion
- * of additional resources could be triggered with
- * <code>includePatterns</code>. When the inclusion patterns is too
- * large, eviction of previously selected resources could be triggered
- * with <code>excludePatterns</code>.
+ * Replaced by {@code quarkus.native.resources.excludes} in Camel
Quarkus 2.0.0.
+ * Using this property throws an exception at build time.
*/
@ConfigItem
public Optional<List<String>> excludePatterns;
/**
- * A comma separated list of Ant-path style patterns to match resources
- * that should be <strong>included</strong> in the native executable.
By default,
- * resources not selected by quarkus itself are ignored. Then,
inclusion
- * of additional resources could be triggered with
- * <code>includePatterns</code>. When the inclusion patterns is too
- * large, eviction of previously selected resources could be triggered
- * with <code>excludePatterns</code>.
+ * Replaced by {@code quarkus.native.resources.includes} in Camel
Quarkus 2.0.0.
+ * Using this property throws an exception at build time.
*/
@ConfigItem
public Optional<List<String>> includePatterns;
Review comment:
Ah, sorry, I have not understood initially. If we remove it, users just
get a warning about an unknown property. With the changes in the Processor
https://github.com/apache/camel-quarkus/pull/2813/files they get a real build
time failure.
##########
File path:
integration-tests/geocoder/src/main/java/org/apache/camel/quarkus/component/geocoder/it/MockApiService.java
##########
@@ -21,8 +21,10 @@
import javax.enterprise.context.ApplicationScoped;
import com.google.maps.GeoApiContext;
+import io.quarkus.runtime.annotations.RegisterForReflection;
@ApplicationScoped
+@RegisterForReflection(targets = GeoApiContext.Builder.class)
Review comment:
Unlike `@RegisterForReflection`, `quarkus.camel.native.reflection`
supports wildcards so they are not fully interchangeable.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
[email protected]
