> On May 13, 2014, 11:07 a.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.
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. - Dominic ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/21386/#review42853 ----------------------------------------------------------- On May 13, 2014, 11:07 a.m., Dominic Hamon wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/21386/ > ----------------------------------------------------------- > > (Updated May 13, 2014, 11:07 a.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 > >