<html>
<head>
<base href="https://bugs.freedesktop.org/" />
</head>
<body>
<p>
<div>
<b><a class="bz_bug_link
bz_status_NEW "
title="NEW - Mesa's demo / tools won't compile since EGL changes"
href="https://bugs.freedesktop.org/show_bug.cgi?id=91020#c5">Comment # 5</a>
on <a class="bz_bug_link
bz_status_NEW "
title="NEW - Mesa's demo / tools won't compile since EGL changes"
href="https://bugs.freedesktop.org/show_bug.cgi?id=91020">bug 91020</a>
from <span class="vcard"><a class="email" href="mailto:emil.l.velikov@gmail.com" title="Emil Velikov <emil.l.velikov@gmail.com>"> <span class="fn">Emil Velikov</span></a>
</span></b>
<pre>(In reply to George Diamantopoulos from <a href="show_bug.cgi?id=91020#c4">comment #4</a>)
<span class="quote">> Created <span class=""><a href="attachment.cgi?id=117106" name="attach_117106" title="Fix build against EGL implementations that don't support the EGL_MESA_screen_surface extension">attachment 117106</a> <a href="attachment.cgi?id=117106&action=edit" title="Fix build against EGL implementations that don't support the EGL_MESA_screen_surface extension">[details]</a></span> <a href='page.cgi?id=splinter.html&bug=91020&attachment=117106'>[review]</a> [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</span >
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</pre>
</div>
</p>
<hr>
<span>You are receiving this mail because:</span>
<ul>
<li>You are the QA Contact for the bug.</li>
<li>You are the assignee for the bug.</li>
</ul>
</body>
</html>