You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@geode.apache.org by Jens Deppe <jd...@pivotal.io> on 2016/02/01 19:56:24 UTC

Re: Review Request 42655: GEODE-830: Allow transitive test dependencies

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

(Updated Feb. 1, 2016, 6:56 p.m.)


Review request for geode, Anthony Baker and Dan Smith.


Changes
-------

Coming back to this after some distractions. Made changes to accommodate the new build refactor. Also changed the way the transitive deps are manipulated. This is not part of a configuration/dependencies Rule any more.


Repository: geode


Description
-------

GEODE-830: Allow transitive test dependencies


Diffs (updated)
-----

  gemfire-core/build.gradle 719435017aaa24168bb70e1a2e329cd4987a473f 
  gemfire-lucene/build.gradle 6218e55c73f771a46a74ae31606ce1ace5b2008f 
  gemfire-rebalancer/build.gradle 1f9bff8d7d37700627ca1b6e8aa74827d59b76d4 
  gradle/dependency-versions.properties 25f297d69db67038a5b4b72ae0c5f44a8a8aafd2 
  gradle/java.gradle 3ec105d7a37fd036527ea6f78c61f7dbcca3244d 
  gradle/test.gradle b076f2611df4a746623aae8d0ca8864a3d18654d 

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


Testing
-------


Thanks,

Jens Deppe


Re: Review Request 42655: GEODE-830: Allow transitive test dependencies

Posted by Mark Bretl <mb...@apache.org>.
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/42655/#review117521
-----------------------------------------------------------


Ship it!




Ship It!

- Mark Bretl


On Feb. 1, 2016, 10:56 a.m., Jens Deppe wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/42655/
> -----------------------------------------------------------
> 
> (Updated Feb. 1, 2016, 10:56 a.m.)
> 
> 
> Review request for geode, Anthony Baker and Dan Smith.
> 
> 
> Repository: geode
> 
> 
> Description
> -------
> 
> GEODE-830: Allow transitive test dependencies
> 
> 
> Diffs
> -----
> 
>   gemfire-core/build.gradle 719435017aaa24168bb70e1a2e329cd4987a473f 
>   gemfire-lucene/build.gradle 6218e55c73f771a46a74ae31606ce1ace5b2008f 
>   gemfire-rebalancer/build.gradle 1f9bff8d7d37700627ca1b6e8aa74827d59b76d4 
>   gradle/dependency-versions.properties 25f297d69db67038a5b4b72ae0c5f44a8a8aafd2 
>   gradle/java.gradle 3ec105d7a37fd036527ea6f78c61f7dbcca3244d 
>   gradle/test.gradle b076f2611df4a746623aae8d0ca8864a3d18654d 
> 
> Diff: https://reviews.apache.org/r/42655/diff/
> 
> 
> Testing
> -------
> 
> 
> Thanks,
> 
> Jens Deppe
> 
>


Re: Review Request 42655: GEODE-830: Allow transitive test dependencies

Posted by Dan Smith <ds...@pivotal.io>.
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/42655/#review117517
-----------------------------------------------------------


Ship it!




Ship It!

- Dan Smith


On Feb. 1, 2016, 6:56 p.m., Jens Deppe wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/42655/
> -----------------------------------------------------------
> 
> (Updated Feb. 1, 2016, 6:56 p.m.)
> 
> 
> Review request for geode, Anthony Baker and Dan Smith.
> 
> 
> Repository: geode
> 
> 
> Description
> -------
> 
> GEODE-830: Allow transitive test dependencies
> 
> 
> Diffs
> -----
> 
>   gemfire-core/build.gradle 719435017aaa24168bb70e1a2e329cd4987a473f 
>   gemfire-lucene/build.gradle 6218e55c73f771a46a74ae31606ce1ace5b2008f 
>   gemfire-rebalancer/build.gradle 1f9bff8d7d37700627ca1b6e8aa74827d59b76d4 
>   gradle/dependency-versions.properties 25f297d69db67038a5b4b72ae0c5f44a8a8aafd2 
>   gradle/java.gradle 3ec105d7a37fd036527ea6f78c61f7dbcca3244d 
>   gradle/test.gradle b076f2611df4a746623aae8d0ca8864a3d18654d 
> 
> Diff: https://reviews.apache.org/r/42655/diff/
> 
> 
> Testing
> -------
> 
> 
> Thanks,
> 
> Jens Deppe
> 
>