Community
Participate
Working Groups
We want to update the pom version tag in all child modules to be the same as in the manifest. As the pomgenerator is using the manifest-attributes to generate poms it would be nice to have an relativePath command parameter or have an approach where an existing parent pom would be used. Our parent pom is in a separate folder (e.g ../<path to project>/builder/pom.xml), so I can't just generate new child poms and replace the parent pom. At the moment I need to add the relativePath to every child pom manually. Doing so within the plugin would be very helpful. I have a .patch for the relativePath command parameter, as this is just minor changes to the code. See attached file.´
Created attachment 198179 [details] .patch for relativePath parameter
I am not sure I understand the usecase. Do you want to update pom.xml versions to match bundle manifests?
Sorry for the late response. Yes the major usecase is to update pom.xml versions to match bundle manifest.
Ok, I understand the usecase, but I believe this functionality belongs to a new goal in tycho-versions-plugin. tycho-pomgenerator-plugin is for one-time initial generation of pom.xml files.
Comment on attachment 198179 [details] .patch for relativePath parameter The proposed patch falls short of the requirements: It is not sufficient to just override the parent pom path - the POM at that location may not match the specified GAV.
It may be a valid use case to generate POMs using an existing parent POM, but since this can be quite tricky to get right, I'm not sure that we want to pursue this idea at all. For now, I changed the title to match your concrete requirement: was: "tycho-pomgenerator-plugin should have a parentPom/relativePath command parameter", now: "Provide a tool to reset POM versions to match manifest versions" As Igor already proposed, this should probably be implemented as a new goal of the versions plugin. It would be great if you could propose another patch :-)
Yeah. Seems like the best idea. I'll see if I can find enough time to provide another patch. :-) Though it could take some time.
Implemented. http://git.eclipse.org/c/tycho/org.eclipse.tycho.git/commit/?id=2487379d5bbec28aca1138eeca37132705fe11a5