[GitHub] [camel-quarkus] lburgazzoli commented on a change in pull request #1984: Fixing core build because of recent changes in camel master

[GitBox <gi...@apache.org>]

lburgazzoli commented on a change in pull request #1984:
URL: https://github.com/apache/camel-quarkus/pull/1984#discussion_r517448828



##########
File path: extensions-core/core/deployment/src/main/java/org/apache/camel/quarkus/core/deployment/CamelProcessor.java
##########
@@ -346,4 +347,10 @@ void unremovableBeans(BuildProducer<AdditionalBeanBuildItem> beanProducer) {
                 .map(CamelRoutesBuilderClassBuildItem::new)
                 .collect(Collectors.toList());
     }
+
+    @BuildStep
+    NativeImageResourceBuildItem initResources() {
+        return new NativeImageResourceBuildItem(
+                "META-INF/services/org/apache/camel/bean-processor-factory");

Review comment:
       afaik those files are scanned and the related classes are added to the FastFactoryFinder so it should not be required to include the resource, see: https://github.com/apache/camel-quarkus/blob/master/extensions-core/core/deployment/src/main/java/org/apache/camel/quarkus/core/deployment/CamelProcessor.java#L136-L159
   
   




----------------------------------------------------------------
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:
users@infra.apache.org