> On Dec. 17, 2014, 6:40 p.m., Joe Smith wrote: > > Maybe this is actually legitimate tho? Perhaps we do want to catch PEX-INFO > > formatting changes?
True -- this is really a bug in pex. - Brian ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/29132/#review65340 ----------------------------------------------------------- On Dec. 17, 2014, 12:57 a.m., Brian Wickman wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/29132/ > ----------------------------------------------------------- > > (Updated Dec. 17, 2014, 12:57 a.m.) > > > Review request for Aurora and Kevin Sweeney. > > > Repository: aurora > > > Description > ------- > > get_client_version should never be allowed to run unpatched in test. > > > Diffs > ----- > > src/test/python/apache/aurora/client/cli/test_help.py > 1576edcaf0f59911c1c7aec42052d5ebfe41caac > > Diff: https://reviews.apache.org/r/29132/diff/ > > > Testing > ------- > > get_client_version tries to get the PEX-INFO of sys.argv[0], which is usually > pants when running tests (!) if the format of PEX-INFO changes to a new > version of pants (as is the case for pants 0.0.25), then tests will start > failing with an inscrutible error. instead, just patch get_client_version to > something sensible. > > > Thanks, > > Brian Wickman > >
