laurentschoelens commented on PR #1445:
URL: https://github.com/apache/cxf/pull/1445#issuecomment-1729484402

   > > * B will then have access and depends really on A (you can write code 
with classes under A)
   > 
   > Correct, but if the dependency is not available at runtime, the 
application will fail (if is not mitigated)
   
   Here having it with default scope but only Optional will bring it on runtime 
by default when execution B as application.
   
   > > Here, codegen does have spring dependency as direct dependency, even if 
marked as optional (which has no effect cause final artifact). Another fix 
would be to check if codegen really depends on spring classes (inside his own 
module) and remove it if nod needed.
   > 
   > The optional part indeed was not sufficient hence the `provided` was 
added, those are never brought up by default. `codegen` does not depend on the 
Spring classes directly.
   
   Doing so you'll always get the failing "ClassNotFoundException" with no more 
usage of SpringBusFactory on plugins.
   Don't know if that's excepted...
   It'll only use SpringBusFactory class only by adding manually Spring as 
dependency in the plugin...


-- 
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: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]

Reply via email to