[GitHub] [camel-quarkus] essobedo commented on a diff in pull request #4217: Fix #4216 to make xml integration tests working in native build

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

essobedo commented on code in PR #4217:
URL: https://github.com/apache/camel-quarkus/pull/4217#discussion_r1006512665


##########
extensions/xslt/deployment/src/main/java/org/apache/camel/quarkus/component/xslt/deployment/XsltProcessor.java:
##########
@@ -149,4 +150,10 @@ void xsltResources(
         }
     }
 
+    //It should be removed with fixing https://github.com/apache/camel-quarkus/issues/1615
+    @BuildStep
+    void runtimeInit(BuildProducer<RuntimeInitializedClassBuildItem> deps) {
+        deps.produce(new RuntimeInitializedClassBuildItem("org.apache.hc.client5.http.impl.auth.NTLMEngineImpl"));

Review Comment:
   Please note that since you have only one `RuntimeInitializedClassBuildItem` to produce, you could omit the producer and just return your instance of `RuntimeInitializedClassBuildItem`, something like:
   ```suggestion
       RuntimeInitializedClassBuildItem runtimeInit() {
           return new RuntimeInitializedClassBuildItem("org.apache.hc.client5.http.impl.auth.NTLMEngineImpl");
   ```



-- 
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.

To unsubscribe, e-mail: commits-unsubscribe@camel.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org