Mailing-List: contact cloudstack-dev-help@incubator.apache.org; run by ezmlm
Precedence: bulk
Reply-To: cloudstack-dev@incubator.apache.org
Received-SPF: pass (nike.apache.org: domain of kelven.yang@citrix.com
 designates 66.165.176.63 as permitted sender)
From: Kelven Yang <kelven.yang@citrix.com>
To: "cloudstack-dev@incubator.apache.org"
	<cloudstack-dev@incubator.apache.org>
Date: Fri, 24 Aug 2012 11:59:09 -0700
Subject: Re: maven patch
Thread-Topic: maven patch
Thread-Index: Ac2CKotKTSxSm7NJRWWC+hClJMZtbw==
Message-ID: <CC5D1CA3.CAA2%kelven.yang@citrix.com>
In-Reply-To: 
 <20120824113710.d3d18d9a633cb81ed61112bf108fc615.a7a48e3164.wbe@email00.secureserver.net>
Accept-Language: en-US
Content-Language: en-US
user-agent: Microsoft-MacOutlook/14.2.3.120616
acceptlanguage: en-US
Content-Type: text/plain; charset="us-ascii"
Content-Transfer-Encoding: quoted-printable
MIME-Version: 1.0

I'll do the merge to master branch this afternoon

Kelven

On 8/24/12 11:37 AM, "Darren Shepherd" <darren@godaddy.com> wrote:

>I have the integrations with ant/waf done also, but I need the maven
>branch merge to master first so that I can submit the next round of
>patches.  The problem though is the last two commits on the maven branch
>have deleted all the Eclipse projects files.  While this is definitely
>the right thing to do, I didn't want to do this right now.
>
>For the first round of maven integration I'm trying to do it in a
>fashion that has as little impact on the current CS developers and their
>dev environment.  Once we have mvn building then we can go and delete
>the Eclipse files and the binaries in deps and force all developers to
>update their Eclipse environments to M2E.
>
>Can somebody merge the maven branch minus the last two commits to
>master?
>
>Darren
>
>
>
>> -------- Original Message --------
>> Subject: RE: maven patch
>> From: Will Chan <will.chan@citrix.com>
>> Date: Fri, August 24, 2012 9:42 am
>> To: "cloudstack-dev@incubator.apache.org"
>> <cloudstack-dev@incubator.apache.org>
>>=20
>>=20
>> +1 on documentation (also not volunteering as I barely known maven as
>>well).
>>=20
>> > -----Original Message-----
>> > From: John Kinsella [mailto:jlk@stratosec.co]
>> > Sent: Friday, August 24, 2012 9:20 AM
>> > To: <cloudstack-dev@incubator.apache.org>
>> > Subject: Re: maven patch
>> >=20
>> > would be nice to document this in the wiki as well... (not
>>volunteering, I
>> > barely know maven )
>> >=20
>> > On Aug 24, 2012, at 5:02 AM, Hugo Trippaers
>> > <HTrippaers@schubergphilis.com<mailto:HTrippaers@schubergphilis.com>
>> > > wrote:
>> >=20
>> > +1
>> >=20
>> > Darren any thoughts on the next step? Would be nice to integrate
>>further
>> > with waf etc.
>> >=20
>> > Cheers,
>> >=20
>> > Hugo
>> >=20
>> > -----Original Message-----
>> > From: Kelven Yang [mailto:kelven.yang@citrix.com<http://citrix.com>]
>> > Sent: Thursday, August 23, 2012 11:37 PM
>> > To: cloudstack-dev@incubator.apache.org<mailto:cloudstack-
>> > dev@incubator.apache.org>
>> > Subject: Re: maven patch
>> >=20
>> > 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.
>> >=20
>> > 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
>> >=20
>> > Kelven
>> >=20
>> > On 8/23/12 1:44 PM, "David Nalley"
>> > <david@gnsa.us<mailto:david@gnsa.us>> wrote:
>> >=20
>> > On Thu, Aug 23, 2012 at 11:20 AM, Darren Shepherd
>> > <darren@godaddy.com<mailto:darren@godaddy.com>>
>> > wrote:
>> > Some Committer,
>> >=20
>> > 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.
>> >=20
>> > Darren
>> >=20
>> >=20
>> > Merge request against the github mirror should work as well, and is
>>likely
>> > less work.
>> >=20
>> > --David
>> >=20
>> >=20
>> >=20
>> > Stratosec<http://stratosec.co> - Secure Infrastructure as a Service
>> > o: 415.315.9385
>> > @johnlkinsella<http://twitter.com/johnlkinsella>