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


No functional change, I just promised a follow-up to 
https://reviews.apache.org/r/40299/ with another approach and this is it.
This is a take-it or leave it.  It certainly could be see as obscuring if folks 
are used to looking in pants.ini for a custom repos setup.

- John Sirois


On Nov. 14, 2015, 5:18 p.m., John Sirois wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/40324/
> -----------------------------------------------------------
> 
> (Updated Nov. 14, 2015, 5:18 p.m.)
> 
> 
> Review request for Aurora, Joshua Cohen and Bill Farner.
> 
> 
> Bugs: AURORA-1538
>     https://issues.apache.org/jira/browse/AURORA-1538
> 
> 
> Repository: aurora
> 
> 
> Description
> -------
> 
> This lifts `mesos.interface` and `mesos.native` out of
> `requirements.txt`, ties their versions together and isolates the
> custom repository needed by `mesos.native` to its `python_requirement`.
> 
>  3rdparty/python/BUILD            | 26 ++++++++++++++++++++++++++
>  3rdparty/python/requirements.txt |  2 --
>  pants.ini                        |  5 -----
>  3 files changed, 26 insertions(+), 7 deletions(-)
> 
> 
> Diffs
> -----
> 
>   3rdparty/python/BUILD 7ef81d13afa0089d7e8a779e71b53e0fc1848466 
>   3rdparty/python/requirements.txt 3a78b754ad8e55308554119e279693053951cc6e 
>   pants.ini 319d38e9a7af8055cac5bbce4a6ae0cbb38dc8d0 
> 
> Diff: https://reviews.apache.org/r/40324/diff/
> 
> 
> Testing
> -------
> 
> Successfully ran end-to-end with this change using a fresh vagrant
> image and clean repo via:
> ```
> vagrant destroy && \
> bash src/test/sh/org/apache/aurora/e2e/test_end_to_end.sh
> ```
> 
> 
> Thanks,
> 
> John Sirois
> 
>

Reply via email to