> On Dec. 8, 2016, 8:29 p.m., John Sirois wrote: > > build-support/thrift/prepare_binary.sh, line 29 > > <https://reviews.apache.org/r/54550/diff/3/?file=1580209#file1580209line29> > > > > aha - popd on linux echos the dirstack - presumably it does not do this > > on osx... so needs a re-direct. Ditto pushd. And AuroraBot really does > > need to go green. > > John Sirois wrote: > ...where linux probably just means modern bash, osx - even Sierra IIUC, > is wtfb (way tf behind).
OK - I've reproed the failure mode on my linux machine. I'll send up an RB to fix. I would love to get to the bottom of the pushd/popd thrash though since it seems like this is not needed on linux - which makes no sense to me yet. - John ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/54550/#review158632 ----------------------------------------------------------- On Dec. 8, 2016, 7:30 p.m., Joshua Cohen wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/54550/ > ----------------------------------------------------------- > > (Updated Dec. 8, 2016, 7:30 p.m.) > > > Review request for Aurora and Santhosh Kumar Shanmugham. > > > Repository: aurora > > > Description > ------- > > We have some CI machines internally where the default python doesn't have the > `json` module, so this script fails. > > > Diffs > ----- > > build-support/thrift/prepare_binary.sh > a96b33e4564604f72d8fcf284f0cf1adea1866a9 > > Diff: https://reviews.apache.org/r/54550/diff/ > > > Testing > ------- > > > Thanks, > > Joshua Cohen > >