Ned Deily added the comment:

I took a quick look at the revised patch.  Clearly, it's too big to review at a 
detailed level.  I tried the approach of running two complete verbose regrtest 
runs (-m test -v -uall), before and after the patch, and diffing the results in 
a "smart" diff app.  Again, this is extremely tedious: while some simple 
filtering could be done to mask some differences (like elapsed times and time 
stamps), the non-deterministic output of many tests remains (and demonstrates 
how difficult it currently is to automatically compare regrtest run results).  
I did a spot check of the results and didn't notice any glaring problems and 
*did* notice that some tests were now being reported as skipped, as expected.  
Overall, I think this is a useful change to the test suite so I would say apply 
it to default (for 3.4.0) but I'm ambivalent about applying it to 3.3 at this 
point.

----------
nosy: +ned.deily

_______________________________________
Python tracker <rep...@bugs.python.org>
<http://bugs.python.org/issue18702>
_______________________________________
_______________________________________________
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

Reply via email to