https://bugs.freedesktop.org/show_bug.cgi?id=91020

--- Comment #5 from Emil Velikov <emil.l.veli...@gmail.com> ---
(In reply to George Diamantopoulos from comment #4)
> Created attachment 117106 [details] [review]
> Fix build against EGL implementations that don't support  the
> EGL_MESA_screen_surface extension
> 
> There is a patch by Frank Binns (posted on mesa-dev) since 2012 on this...
> It worked for me, so I'm attaching it here

While this should work and has been the approach used so far in mesa-demos, I'm
having second doubts how wise of an idea it is.

In this particular example, this will result in a "empty" demo/program which
won't do anything when run on a EGL_MESA_screen_surface capable driver.

I.e. even if your system is perfectly fine, the binary build by your
distro/elsewhere with the missing definition will be useless.

May I suggest that we add the relevant defines, after including the *GL
headers. This way the binary will work on compatible systems, and (should) bail
out if the extension is not supported.

One could also fix the other similar cases, while at it ;-) Git grep shows some
~120 lines :-P

-- 
You are receiving this mail because:
You are the QA Contact for the bug.
You are the assignee for the bug.
_______________________________________________
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/mesa-dev

Reply via email to