> On Feb. 4, 2014, 12:34 a.m., Brian Wickman wrote:
> > pants, line 20
> > <https://reviews.apache.org/r/17679/diff/1/?file=463687#file463687line20>
> >
> >     why not PANTS_VERSION=$(cat .pantsversion) like before?
> 
> Kevin Sweeney wrote:
>     This matches the convention in other virtualenv-y scripts, but I don't 
> have strong feelings one way or another.

ok!


- Brian


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


On Feb. 3, 2014, 11:13 p.m., Kevin Sweeney wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/17679/
> -----------------------------------------------------------
> 
> (Updated Feb. 3, 2014, 11:13 p.m.)
> 
> 
> Review request for Aurora and Brian Wickman.
> 
> 
> Repository: aurora
> 
> 
> Description
> -------
> 
> Upgrade twitter.common.python to 0.3.0
> 
> Use the new "pex" utility from twitter.common.python to bootstrap pants
> rather than relying on knowledge of the inner workings of
> twitter.common.python APIs. I also took the opportunity to refactor the
> common "maybe bootstrap a virtualenv" pattern used with some support
> tools to use a consistent build-support/NAME.venv pattern.
> 
> 
> Diffs
> -----
> 
>   .gitignore 25114b9fe41747b60de2d176f959ff26571f0062 
>   build-support/pex PRE-CREATION 
>   build-support/release/make-python-sdists 
> b2314d72249fee81e434a220a75cbc8588495c5a 
>   pants 229e201b8fa7b069eee4291695ce341318ef976e 
>   rbt 8828ead987ba84ef8d7435b30b7ff99a1d5e20a1 
> 
> Diff: https://reviews.apache.org/r/17679/diff/
> 
> 
> Testing
> -------
> 
> Ran ./pants, ./rbt, ./build-support/pex, 
> ./build-support/release/make-python-sdists from clean and dirty repos.
> 
> ./pants src/test/python:all
> 
> 
> Thanks,
> 
> Kevin Sweeney
> 
>

Reply via email to