You are viewing a plain text version of this content. The canonical link for it is here.

Posted to commits@camel.apache.org by GitBox <gi...@apache.org> on 2020/11/04 09:30:53 UTC

[GitHub] [camel] astefanutti commented on a change in pull request #4562: [CAMEL-15725] setting up basic OSGi-CDI metadata in the manifest

astefanutti commented on a change in pull request #4562:
URL: https://github.com/apache/camel/pull/4562#discussion_r517207009



##########
File path: components/camel-cdi/src/main/java/org/apache/camel/cdi/AnnotatedDelegate.java
##########
@@ -21,11 +21,13 @@
 import java.util.HashSet;
 import java.util.Set;
 
+import javax.enterprise.inject.Vetoed;

Review comment:
       IIRC, we did not used the `@Vetoed` annotation to maintain compatibility with CDI 1.0.
   
   It works because the default profile `weld-3.0` uses CDI 2.0, but other profiles may fail.
   
   That being said, we may want to consider dropping CDI 1.0 support.




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