> On May 13, 2014, 6:07 p.m., Bill Farner wrote:
> > build-support/tools/list-missing-shipits, line 94
> > <https://reviews.apache.org/r/21386/diff/1/?file=580750#file580750line94>
> >
> >     At a quick glance, it's not obvious how this relates to plumbing the 
> > 'project' argument.  Was there another issue you bumped into?  Is this so 
> > you can use the script outside the repo?
> 
> Dominic Hamon wrote:
>     yes - other projects don't have rbt in the specific path the script was 
> pointing at.
>     
>     I considered keeping this as the default if 'rbt' isn't found on the path 
> but thought that might be flaky.
> 
> Kevin Sweeney wrote:
>     FWIW I'd highly recommend the aurora approach (self-bootstrapping, 
> isolated rbt) over requiring a global installation of it with pip, since you 
> get to explicitly specify the target version and it's one less barrier to 
> entry for new contributors. To get this in another repo you'd want to copy 
> ./rbt, ./build-support/virtualenv, and 
> ./build-support/tools/list-missing-shipits.
> 
> Dominic Hamon wrote:
>     that sounds like a great plan, especially with all the recent churn on 
> our own post-review script. Perhaps we should compare notes on each approach.

Agree with Kevin on this, we should keep the set path for our version of rbt 
which runs within venv and bootstraps itself (this helps avoid issues like 
Mesos ran into with the latest version of rbt). It might be worth looking into 
as a sub command within rbt to avoid having to call externally


- Jake


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


On May 13, 2014, 6:07 p.m., Dominic Hamon wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/21386/
> -----------------------------------------------------------
> 
> (Updated May 13, 2014, 6:07 p.m.)
> 
> 
> Review request for Aurora, Kevin Sweeney and Bill Farner.
> 
> 
> Repository: aurora
> 
> 
> Description
> -------
> 
> Add support for custom project to list-missing-shipits.
> Search for rbt on PATH.
> 
> 
> Diffs
> -----
> 
>   build-support/tools/list-missing-shipits 
> 33a4c412092a5a9a21ed8e5aaee69989535a12e7 
> 
> Diff: https://reviews.apache.org/r/21386/diff/
> 
> 
> Testing
> -------
> 
> Ran script against project Mesos.
> 
> 
> Thanks,
> 
> Dominic Hamon
> 
>

Reply via email to