[PATCH weston] tests: Only run buffer-count test on Mesa >= 10.1
Eoff, Ullysses A
ullysses.a.eoff at intel.com
Wed Jan 8 10:22:20 PST 2014
Indeed, this looks better to me... I've confirmed with my testing, too, that we need Mesa >= 10.1
> -----Original Message-----
> From: wayland-devel-bounces at lists.freedesktop.org [mailto:wayland-devel-bounces at lists.freedesktop.org] On Behalf Of
> ppaalanen at gmail.com
> Sent: Wednesday, January 08, 2014 6:06 AM
> To: krh at bitplanet.net
> Cc: Pekka Paalanen; wayland-devel at lists.freedesktop.org
> Subject: [PATCH weston] tests: Only run buffer-count test on Mesa >= 10.1
>
> From: Pekka Paalanen <pekka.paalanen at collabora.co.uk>
>
> Comment #2 in the bug report says Mesa 10.0 branch does not have the
> fix, and indeed buffer-count test fails on Mesa 10.0.1. Fix the test to
> require Mesa 10.1 or later.
>
> Now I correctly get:
> mesa version too old (OpenGL ES 3.0 Mesa 10.0.1 (git-12484d2))
>
> https://bugs.freedesktop.org/show_bug.cgi?id=72835
>
> Signed-off-by: Pekka Paalanen <pekka.paalanen at collabora.co.uk>
> ---
> tests/buffer-count-test.c | 4 ++--
> 1 file changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/tests/buffer-count-test.c b/tests/buffer-count-test.c
> index 3b66e31..c7ddeb6 100644
> --- a/tests/buffer-count-test.c
> +++ b/tests/buffer-count-test.c
> @@ -97,7 +97,7 @@ init_egl(struct test_data *test_data)
> test_data->egl_surface, test_data->egl_ctx);
> assert(ret == EGL_TRUE);
>
> - /* This test is specific to mesa 10.0 and later, which is the
> + /* This test is specific to mesa 10.1 and later, which is the
> * first release that doesn't accidentally triple-buffer. */
> str = (const char *) glGetString(GL_VERSION);
> mesa = strstr(str, "Mesa ");
> @@ -105,7 +105,7 @@ init_egl(struct test_data *test_data)
> skip("unknown EGL implementation (%s)\n", str);
> if (sscanf(mesa + 5, "%d.%d", &major, &minor) != 2)
> skip("unrecognized mesa version (%s)\n", str);
> - if (major < 10)
> + if (major < 10 || (major == 10 && minor < 1))
> skip("mesa version too old (%s)\n", str);
>
> }
> --
> 1.8.3.2
>
> _______________________________________________
> wayland-devel mailing list
> wayland-devel at lists.freedesktop.org
> http://lists.freedesktop.org/mailman/listinfo/wayland-devel
More information about the wayland-devel
mailing list