On 02/27/2012 03:00 PM, Vinson Lee wrote: > On Mon, Feb 27, 2012 at 6:57 AM, Lucas Meneghel Rodrigues > <[email protected]> wrote: >> >> I suggest a slight change on the patch, to actually make a list of the >> failed tests and then use that list to throw the TestFail exception. I've >> sent an updated version of the patch (labeled as v3), would you please take >> a look at it and ACK it if you like the approach? >> > > Hi, Lucas. > > The patch looks good to me, and I have ACK'ed it. > > I'm interested in getting the bug fixed, but don't have a strong > preference for any approach. There were some other ideas that I > considered too. One was to use failcmdfile, which lists the failed > tests. Another was to capture the output (runltp -o option) and parse > the stat line.
Ok, it's a done deal then: https://github.com/autotest/autotest/commit/72444e070384a59c68f1629c11dfa57f0831c6c2 Thanks! > Cheers, > Vinson _______________________________________________ Autotest mailing list [email protected] http://test.kernel.org/cgi-bin/mailman/listinfo/autotest
