==System Environment== -------------------------- Piglit: 534e6a8585ae6bd41148014d8d68c9018d7f94e4 Libdrm: (master)libdrm-2.4.58-4-g00847fa48b83a85b0cb882594a12ed1511f780db Mesa: (master)8cf6482c3da9493dacf4a6a495ae3c0193cf75a7 Xserver: (master)xorg-server-1.16.0-418-g61a292adf45405641de1c522a04c148e0a152acd Xf86_video_intel:(master)2.99.916-98-g49cf2803d3bc3be12f6a59581631fdf5547bb0e3 Libva: (master)ccd93de5a707e92a629cccd595757c8d436fa3cc Libva_intel_driver:(master)f11176415ec26eb5960ba6841d2d9c22f2cabc60 Kernel: (drm-intel-nightly)2ea23cd593ba60ead60e2f796fae675aa4475b1a ==Bug detailed description== ----------------------------- Run piglit case report: Traceback (most recent call last): File "./piglit-print-commands.py", line 34, in <module> from framework.exectest import Test ImportError: No module named exectest test thread complete Bisect shows: 3d00e91606e06b1c74a3d479176f6ddc0de19db2 is the first bad commit commit 3d00e91606e06b1c74a3d479176f6ddc0de19db2 Author: Dylan Baker <dylanx.c.baker@intel.com> AuthorDate: Wed Sep 10 10:21:25 2014 -0700 Commit: Dylan Baker <dylanx.c.baker@intel.com> CommitDate: Tue Oct 14 14:33:03 2014 -0700 framework: Move tests derived classes into a package This moves all of the classes that derive from Test into a package called test. This patch looks very large, but is mostly just moving files around and changing imports to account for this. So why all of this code churn? The big advantage is that through __init__.py magic there is one module to be imported, framework.test. This module contains all of the public classes, functions, and constants, in one place while hiding the rest, but allowing them to be accessed explicitly. Signed-off-by: Dylan Baker <dylanx.c.baker@intel.com> ==Reproduce steps== ---------------------------- 1. Run all Piglit cases
Created attachment 108494 [details] [review] A small patch used to fix this issue I just modify the piglit test as attachment, then it can be worked.
Your patch looks correct. It could be simplified a little bit, and I've sent a slightly tweaked version to the list. http://patchwork.freedesktop.org/patch/35716/
(In reply to Dylan from comment #2) > Your patch looks correct. It could be simplified a little bit, and I've sent > a slightly tweaked version to the list. > > http://patchwork.freedesktop.org/patch/35716/ Fixed by this patch.
Fixed To ssh://git.fdo/git/piglit 05cd842..3c68657 master -> master
Fixed.
Use of freedesktop.org services, including Bugzilla, is subject to our Code of Conduct. How we collect and use information is described in our Privacy Policy.