Re: Review Request 24238: Fix mvn marvin.sync profile

2014-08-13 Thread Rohit Yadav
> On Aug. 4, 2014, 4:35 p.m., Santhosh Edukulla wrote: > > tools/marvin/setup.py, line 35 > > > > > > Please tying like this to as a dependency on pom.xml, may unnecessarily > > lead to tight coupling with mvn logic in

Re: Review Request 24238: Fix mvn marvin.sync profile

2014-08-12 Thread Leo Simons
> On Aug. 4, 2014, 4:35 p.m., Santhosh Edukulla wrote: > > tools/marvin/setup.py, line 35 > > > > > > Please tying like this to as a dependency on pom.xml, may unnecessarily > > lead to tight coupling with mvn logic in

Re: Review Request 24238: Fix mvn marvin.sync profile

2014-08-12 Thread Rohit Yadav
> On Aug. 4, 2014, 4:35 p.m., Santhosh Edukulla wrote: > > tools/marvin/setup.py, line 35 > > > > > > Please tying like this to as a dependency on pom.xml, may unnecessarily > > lead to tight coupling with mvn logic in

Re: Review Request 24238: Fix mvn marvin.sync profile

2014-08-05 Thread Leo Simons
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/24238/#review49575 --- tools/marvin/pom.xml

Re: Review Request 24238: Fix mvn marvin.sync profile

2014-08-05 Thread Leo Simons
> On Aug. 4, 2014, 4:35 p.m., Santhosh Edukulla wrote: > > tools/marvin/setup.py, line 35 > > > > > > Please tying like this to as a dependency on pom.xml, may unnecessarily > > lead to tight coupling with mvn logic in

Re: Review Request 24238: Fix mvn marvin.sync profile

2014-08-05 Thread Leo Simons
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/24238/ --- (Updated Aug. 5, 2014, 8:43 a.m.) Review request for cloudstack and Rohit Yadav

Re: Review Request 24238: Fix mvn marvin.sync profile

2014-08-04 Thread Rohit Yadav
> On Aug. 4, 2014, 4:35 p.m., Santhosh Edukulla wrote: > > tools/marvin/setup.py, line 35 > > > > > > Please tying like this to as a dependency on pom.xml, may unnecessarily > > lead to tight coupling with mvn logic in

Re: Review Request 24238: Fix mvn marvin.sync profile

2014-08-04 Thread Leo Simons
> On Aug. 4, 2014, 4:35 p.m., Santhosh Edukulla wrote: > > tools/marvin/setup.py, line 35 > > > > > > Please tying like this to as a dependency on pom.xml, may unnecessarily > > lead to tight coupling with mvn logic in

Re: Review Request 24238: Fix mvn marvin.sync profile

2014-08-04 Thread Santhosh Edukulla
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/24238/#review49460 --- tools/marvin/setup.py

Review Request 24238: Fix mvn marvin.sync profile

2014-08-04 Thread Leo Simons
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/24238/ --- Review request for cloudstack and Rohit Yadav. Repository: cloudstack-git Des