I got Darren's maven patch work nicely in my local environment. Just to be cautious, I pushed it to ASF/maven branch for review first. If I don't hear no no's by tomorrow, I'll push it to master.
After this patch, ant build and maven build will work side by side, this will give us a graceful period to have everyone migrate to maven based build system. If you use Eclipse, you will need to re-import all maven eclipse projects Kelven On 8/23/12 1:44 PM, "David Nalley" <[email protected]> wrote: >On Thu, Aug 23, 2012 at 11:20 AM, Darren Shepherd <[email protected]> >wrote: >> Some Committer, >> >> I can't seem to get the maven patch to work through Review Board. Can a >> committer just pull the "maven" branch from >> https://github.com/ibuildthecloud/incubator-cloudstack.git merge to >> master and commit? One of the problems with the patch is that it moves >> deps/XenServerJava/com to deps/XenServerJava/src/com. With a patch that >> shows as a delete and add, not a move. Besides moving that directory >> the patch is non-impacting in that it just adds the pom.xml files >> everywhere. The summary of the diff will show that its just a bunch of >> adds. >> >> Darren > > >Merge request against the github mirror should work as well, and is >likely less work. > >--David
