> On Nov. 4, 2014, 3:53 p.m., Mark Michelson wrote: > > Why the change to being a standalone script? I personally like the idea of > > it being a filter more.
I changed it locally months back for 2 reasons... The first was that runtests wasn't pre-calculating and printing the total number of tests to run back then. So I had to run runtests twice, once to get the total number (in dry-run mode) then again for real. The other one was buffering problems. ./runtests.py | pretty_print wasn't printing output as often as it should so I was always doing python -u ./runtests.py | pretty_print. Honestly, I didn't think anyone else was using it. :) I can change it back to a filter if that's more useful. - George ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviewboard.asterisk.org/r/4145/#review13673 ----------------------------------------------------------- On Nov. 3, 2014, 8:43 a.m., George Joseph wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviewboard.asterisk.org/r/4145/ > ----------------------------------------------------------- > > (Updated Nov. 3, 2014, 8:43 a.m.) > > > Review request for Asterisk Developers. > > > Repository: testsuite > > > Description > ------- > > Updated contrib/scripts/pretty_print so it works with recent runtests.py > changes. Also changed it from a filter to a standalone script. > > > Diffs > ----- > > asterisk/trunk/contrib/scripts/pretty_print 5878 > > Diff: https://reviewboard.asterisk.org/r/4145/diff/ > > > Testing > ------- > > > Thanks, > > George Joseph > >
-- _____________________________________________________________________ -- Bandwidth and Colocation Provided by http://www.api-digital.com -- asterisk-dev mailing list To UNSUBSCRIBE or update options visit: http://lists.digium.com/mailman/listinfo/asterisk-dev
