davsclaus commented on PR #21800:
URL: https://github.com/apache/camel/pull/21800#issuecomment-4040815755

   > I think the better strategy than putting a dependencyManagement import of 
the Jackson 2 jackson-bom in parent/pom.xml 
https://github.com/apache/camel/pull/21800/changes#diff-b5a06276719e759fe07dfe6f75d781be5f83d2215179d82bdb195ad035348214R3281-R3291
 would be to put the dependency management import of camel-bom in each of the 
individual modules that cannot be converted yet (camel-jackson, 
camel-jacksonxml, camel-jslt, camel-opensearch, camel-openapi-java, 
camel-jbang-mcp). That would minimize the recognition of Jackson 2 to just 
those components.
   > 
   > Putting the Jackson 2 jackson-bom dependencyManagement entry in camel-bom 
is going to make it apply to all modules when we just want it to apply to a 
select six, and it may lead to situations where Jackson 2 dependencies are 
reintroduced in other modules by accident.
   
   Yes that is a good point Tom. Limit jackson 2 to only those components that 
directly use it.
   


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