[Piglit] [PATCH 1/4] framework: Use WGL by default on Windows.

Dylan Baker baker.dylan.c at gmail.com
Mon Dec 15 11:18:48 PST 2014


On Monday, December 15, 2014 07:12:45 PM Jose Fonseca wrote:
> On 15/12/14 17:24, Dylan Baker wrote:
> > On Monday, December 15, 2014 12:19:15 PM Jose Fonseca wrote:
> >> From: José Fonseca <jfonseca at vmware.com>
> >>
> >> Like done with GLX on Linux.
> >> ---
> >>   tests/util/piglit-framework-gl/piglit_wfl_framework.c | 4 +++-
> >>   1 file changed, 3 insertions(+), 1 deletion(-)
> >>
> >> diff --git a/tests/util/piglit-framework-gl/piglit_wfl_framework.c b/tests/util/piglit-framework-gl/piglit_wfl_framework.c
> >> index 75fbacf..5d0f569 100644
> >> --- a/tests/util/piglit-framework-gl/piglit_wfl_framework.c
> >> +++ b/tests/util/piglit-framework-gl/piglit_wfl_framework.c
> >> @@ -60,8 +60,10 @@ piglit_wfl_framework_choose_platform(const struct piglit_gl_test_config *test_co
> >>   			return WAFFLE_PLATFORM_X11_EGL;
> >>   		}
> >>   #endif
> >> -#ifdef PIGLIT_HAS_GLX
> >> +#if defined(PIGLIT_HAS_GLX)
> >>   		return WAFFLE_PLATFORM_GLX;
> >> +#elif defined(PIGLIT_HAS_WGL)
> >> +		return WAFFLE_PLATFORM_WGL;
> >>   #else
> >>   		fprintf(stderr, "environment var PIGLIT_PLATFORM must be set "
> >>   		        "when piglit is built without GLX support\n");
> >
> > You should probably update the comment here as well.

Either way then.

> 
> Sure.  Though the odds of this error is ever seen on Windows is 
> virtually nil (as on Windows Waffle supports nothing but WGL).  That is, 
> most readers of this message will be people building piglit on Unices 
> without X.
> 
> Jose
> 
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 473 bytes
Desc: This is a digitally signed message part.
URL: <http://lists.freedesktop.org/archives/piglit/attachments/20141215/d8e9ebcd/attachment.sig>


More information about the Piglit mailing list