[Piglit] [PATCH 2/2] framework: Don't try to parse GL/GLSL version from wflinfo.
Jose Fonseca
jfonseca at vmware.com
Mon Dec 7 06:26:00 PST 2015
It's making many assumptions about the wflinfo which are not true.
So completely disable it as a workaround.
Though I wonder if there's really any merit in adding a depending on
wflinfo. IMO, if piglit cares for the advertised GL/GLSL versions, it
should have its own internally utility program to dump every thing needed.
Parsing output from external utilities is begging for trouble.
---
framework/test/opengl.py | 5 +++++
1 file changed, 5 insertions(+)
diff --git a/framework/test/opengl.py b/framework/test/opengl.py
index 29da2d1..0a7b2d7 100644
--- a/framework/test/opengl.py
+++ b/framework/test/opengl.py
@@ -80,6 +80,11 @@ class WflInfo(object):
opts -- arguments to pass to wflinfo other than verbose and platform
"""
+
+ # FIXME: The version parsing below is full of bugs. Disable all
+ # wflinfo invocation until they are addressed
+ raise StopWflinfo('OSError')
+
with open(os.devnull, 'w') as d:
try:
raw = subprocess.check_output(
--
2.5.0
More information about the Piglit
mailing list