On Tue, May 08, 2018 at 10:32:26AM -0700, Dylan Baker wrote: > test_list is an iterator, you can't walk an iterator more than once. To > solve this we use itertools.tee, which creates a shared buffer for the > iterators to draw from. This fixes errors like: > > [000/480] > Traceback (most recent call last): > File "./piglit", line 178, in <module> > main() > File "./piglit", line 174, in main > sys.exit(runner(args)) > File "/home/user/piglit/framework/exceptions.py", line 51, in _inner > func(*args, **kwargs) > File "/home/user/piglit/framework/programs/run.py", line 370, in run > backend.finalize({'time_elapsed': time_elapsed.to_json()}) > File "/home/user/piglit/framework/backends/json.py", line 163, in finalize > assert data['tests'] > AssertionError > > Thanks Tomi for figuring out what was wrong with the original patch! > > CC: Michel Dänzer <mic...@daenzer.net> > CC: Tomi Sarvela <tomi.p.sarv...@intel.com> > CC: "Juan A. Suarez Romero" <jasua...@igalia.com>
Oh, just noticed that (well, actually Tomi told me about this). I've sent another patch which fixes the same issue in a slightly different way. Sorry for the duplication! -- Cheers, Arek _______________________________________________ Piglit mailing list Piglit@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/piglit