close

now, I'm not entirely certain, but I don't think ${url} will be populated correctly as the default. Can you confirm you have tested that?

Aside from that, should it also be used for CVS? If not, maybe a .svn. in the property is appropriate?

The patch format is good though, thanks very much for that.

I had a look at the baseUrl usuage and it was a bit of a hack for my environment so yesterday afternoon I changes the plugin to determine the modification set based on the reactor if the mmaven.cruisecontrol.multiproject=true is set, it was a bit more tricky but got me the result I was looking for. The same code should work for CVS and I tested it on single projects.

Assigning to Vincent as he was planning to cut a release of this. Vincent, please review and apply this patch if you think it is appropriate.

One comment: I wouldn't use pom.context.getMavenSession().getRootContext.getVariable(...). I'm not sure why you need that, but I think it is equivalent to the more readable:
<j:set var="list" value="..." scope="parent" />
<j:forEach items="${list}" ... >

Not working on this so leaving it unassigned for anyone interested to take it up.