cunningt opened a new pull request #6518:
URL: https://github.com/apache/camel/pull/6518


   Giving the changes from https://github.com/apache/camel/pull/6462 another 
try.      The properties in camel-dependencies are synced from camel-parent 
within camel-catalog, but there is not a guarantee that the sync will happen 
because of how maven determines the build order.    I don't believe that the 
module order change here 
https://github.com/apache/camel/commit/71e28ebe69a5c87721bbc63693a884473baf302e#diff-9c5fb3d1b7e3b0f54bc5c4182965c4fe1f9023d449017cece3005d3f90e8e4d8
 fixes the issue - maven's reactor determines the correct build order and 
factors in the dependencies of the modules 
(https://maven.apache.org/guides/mini/guide-multiple-modules.html).     
camel-dependencies has no dependencies currently, so from what I've seen in 
building the project, the reactor views it as a good candidate to run early, 
despite the module ordering.
   
   Moved the sync out of camel-catalog and into to camel-dependencies, which 
generates the camel-dependencies pom.   The change to SyncPropertiesMojo 
attaches the camel-dependencies artifact to the build so that it can be 
consumed by the downstream projects.


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