would be nice to document this in the wiki as well… (not volunteering, I barely know maven )
On Aug 24, 2012, at 5:02 AM, Hugo Trippaers <[email protected]<mailto:[email protected]>> wrote: +1 Darren any thoughts on the next step? Would be nice to integrate further with waf etc. Cheers, Hugo -----Original Message----- From: Kelven Yang [mailto:[email protected]<http://citrix.com>] Sent: Thursday, August 23, 2012 11:37 PM To: [email protected]<mailto:[email protected]> Subject: Re: maven patch 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]<mailto:[email protected]>> wrote: On Thu, Aug 23, 2012 at 11:20 AM, Darren Shepherd <[email protected]<mailto:[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 Stratosec<http://stratosec.co> - Secure Infrastructure as a Service o: 415.315.9385 @johnlkinsella<http://twitter.com/johnlkinsella>
