Possibly missing xcb extension? about egl-registry

[caramelli]

Merged fix: https://gitlab.freedesktop.org/mesa/mesa/-/commit/d6b943addeb72a1309b449cea42f4908b17a75d0
@luke-01: eglQueryString(EGL_NO_DISPLAY, EGL_EXTENSIONS) now returns registered name EGL_EXT_platform_xcb instead of EGL_MESA_platform_xcb

[oddhack]

There's a registered EGL_EXT_platform_xcb. Perhaps the MESA version is an earlier iteration of that? @yshui might know since they seem responsible for the EXT, or @caramelli as they alluded to an earlier version on #112.

[yshui]

EGL_EXT_platform_xcb is implemented as EGL_MESA_platform_xcb in mesa, context: https://gitlab.freedesktop.org/mesa/mesa/-/merge_requests/6474#note_610638

Maybe we should rename it?

[oddhack]

EGL_EXT_platform_xcb is implemented as EGL_MESA_platform_xcb in mesa, context: https://gitlab.freedesktop.org/mesa/mesa/-/merge_requests/6474#note_610638

Maybe we should rename it?

If nobody but Mesa is shipping the extension, ISTM the least amount of work to rename the extension specification to use the MESA name string, rather than require Mesa to export two different name strings for the same thing. We can ask Intel / NVIDIA / AMD though not sure who the right contacts are.

[caramelli]

I could be wrong, but maybe the simplest thing that makes sense is to update the EGL_MESA_platform_xcb name returned by Mesa to be consistent with the EGL registry.
The fix is ​​available here: https://gitlab.freedesktop.org/mesa/mesa/-/merge_requests/16609

[oddhack]

I could be wrong, but maybe the simplest thing that makes sense is to update the EGL_MESA_platform_xcb name returned by Mesa to be consistent with the EGL registry. The fix is ​​available here: https://gitlab.freedesktop.org/mesa/mesa/-/merge_requests/16609

If it's not going to adversely affect existing apps then that's a better answer IMO - the extension looked like it had been out for a while so I was thinking changing the exported name string might break something.

We might want to drop a note in the extension spec about this for future archaeologists.

[oddhack]

@caramelli thanks! Closing this - ISTM problem has been resolved without changing the extension spec.