Hi,

On Tuesday, January 20, 2015, Derek Foreman <[email protected]> wrote:

> This also skips the test when running on the headless backend.


I like this part!


> +       if (strstr(extensions, "EGL_WL_bind_wayland_display") == NULL)
> +               skip("Wayland extensions not present.\n");
>

But not this part. bind_wayland_display is the server-side extension, not
the client.

I'd rather skip on:
  - EGL_KHR_platform_base present but not platform_wayland (see Jonny's
patch series)
  - failure to connect to Wayland display
  - failure to create context or surface

That should eliminate basically all the false positives. Thanks for
tackling this niggle though.

Cheers,
Daniel
_______________________________________________
wayland-devel mailing list
[email protected]
http://lists.freedesktop.org/mailman/listinfo/wayland-devel

Reply via email to