You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@ambari.apache.org by Dmitro Lisnichenko <dl...@hortonworks.com> on 2014/03/27 16:22:30 UTC

Review Request 19726: Duplicate definitions of maven-resources-plugin at ambari-server POM

-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/19726/
-----------------------------------------------------------

Review request for Ambari and Dmytro Sen.


Bugs: AMBARI-5240
    https://issues.apache.org/jira/browse/AMBARI-5240


Repository: ambari


Description
-------

Duplicate definitions of maven-resources-plugin are merged and jdeb definitions are optimised a bit.


Diffs
-----

  ambari-agent/pom.xml 20650fb 
  ambari-client/pom.xml d80850e 
  ambari-client/src/main/package/deb/control/control e37581a 
  ambari-client/src/main/package/deb/control/postinst 21a01fa 
  ambari-client/src/main/package/deb/control/postrm 21a01fa 
  ambari-client/src/main/package/deb/control/posttrm 21a01fa 
  ambari-client/src/main/package/deb/control/preinst 21a01fa 
  ambari-client/src/main/package/deb/control/prerm 21a01fa 
  ambari-project/pom.xml 6d8c88a 
  ambari-server/pom.xml 45c1ca7 
  ambari-views/pom.xml 497982c 
  ambari-views/src/main/package/deb/control/control e37581a 
  ambari-views/src/main/package/deb/control/postinst 21a01fa 
  ambari-views/src/main/package/deb/control/postrm 21a01fa 
  ambari-views/src/main/package/deb/control/posttrm 21a01fa 
  ambari-views/src/main/package/deb/control/preinst 21a01fa 
  ambari-views/src/main/package/deb/control/prerm 21a01fa 
  ambari-web/pom.xml 0b96d92 
  pom.xml eeac17c 

Diff: https://reviews.apache.org/r/19726/diff/


Testing
-------

[INFO] ------------------------------------------------------------------------
[INFO] Reactor Summary:
[INFO] 
[INFO] Ambari Main ....................................... SUCCESS [0.539s]
[INFO] Apache Ambari Project POM ......................... SUCCESS [0.040s]
[INFO] Ambari Web ........................................ SUCCESS [6.254s]
[INFO] Ambari Views ...................................... SUCCESS [3.162s]
[INFO] Ambari Server ..................................... SUCCESS [13:12.485s]
[INFO] Ambari Agent ...................................... SUCCESS [12.297s]
[INFO] Ambari Client ..................................... SUCCESS [0.205s]
[INFO] ------------------------------------------------------------------------
[INFO] BUILD SUCCESS
[INFO] ------------------------------------------------------------------------
[INFO] Total time: 13:35.514s
[INFO] Finished at: Thu Mar 27 18:09:52 FET 2014
[INFO] Final Memory: 22M/262M
[INFO] ------------------------------------------------------------------------


Thanks,

Dmitro Lisnichenko


Re: Review Request 19726: Duplicate definitions of maven-resources-plugin at ambari-server POM

Posted by Dmytro Sen <ds...@hortonworks.com>.
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/19726/#review38743
-----------------------------------------------------------

Ship it!


Deb build successfull, all unit test passed

- Dmytro Sen


On March 27, 2014, 3:22 p.m., Dmitro Lisnichenko wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/19726/
> -----------------------------------------------------------
> 
> (Updated March 27, 2014, 3:22 p.m.)
> 
> 
> Review request for Ambari and Dmytro Sen.
> 
> 
> Bugs: AMBARI-5240
>     https://issues.apache.org/jira/browse/AMBARI-5240
> 
> 
> Repository: ambari
> 
> 
> Description
> -------
> 
> Duplicate definitions of maven-resources-plugin are merged and jdeb definitions are optimised a bit.
> 
> 
> Diffs
> -----
> 
>   ambari-agent/pom.xml 20650fb 
>   ambari-client/pom.xml d80850e 
>   ambari-client/src/main/package/deb/control/control e37581a 
>   ambari-client/src/main/package/deb/control/postinst 21a01fa 
>   ambari-client/src/main/package/deb/control/postrm 21a01fa 
>   ambari-client/src/main/package/deb/control/posttrm 21a01fa 
>   ambari-client/src/main/package/deb/control/preinst 21a01fa 
>   ambari-client/src/main/package/deb/control/prerm 21a01fa 
>   ambari-project/pom.xml 6d8c88a 
>   ambari-server/pom.xml 45c1ca7 
>   ambari-views/pom.xml 497982c 
>   ambari-views/src/main/package/deb/control/control e37581a 
>   ambari-views/src/main/package/deb/control/postinst 21a01fa 
>   ambari-views/src/main/package/deb/control/postrm 21a01fa 
>   ambari-views/src/main/package/deb/control/posttrm 21a01fa 
>   ambari-views/src/main/package/deb/control/preinst 21a01fa 
>   ambari-views/src/main/package/deb/control/prerm 21a01fa 
>   ambari-web/pom.xml 0b96d92 
>   pom.xml eeac17c 
> 
> Diff: https://reviews.apache.org/r/19726/diff/
> 
> 
> Testing
> -------
> 
> [INFO] ------------------------------------------------------------------------
> [INFO] Reactor Summary:
> [INFO] 
> [INFO] Ambari Main ....................................... SUCCESS [0.539s]
> [INFO] Apache Ambari Project POM ......................... SUCCESS [0.040s]
> [INFO] Ambari Web ........................................ SUCCESS [6.254s]
> [INFO] Ambari Views ...................................... SUCCESS [3.162s]
> [INFO] Ambari Server ..................................... SUCCESS [13:12.485s]
> [INFO] Ambari Agent ...................................... SUCCESS [12.297s]
> [INFO] Ambari Client ..................................... SUCCESS [0.205s]
> [INFO] ------------------------------------------------------------------------
> [INFO] BUILD SUCCESS
> [INFO] ------------------------------------------------------------------------
> [INFO] Total time: 13:35.514s
> [INFO] Finished at: Thu Mar 27 18:09:52 FET 2014
> [INFO] Final Memory: 22M/262M
> [INFO] ------------------------------------------------------------------------
> 
> 
> Thanks,
> 
> Dmitro Lisnichenko
> 
>