[Mesa-dev] [RFC] glapi: Build gl_gentable.c only on Darwin

[Emil Velikov]

On 9 December 2015 at 14:11, Andreas Boll <andreas.boll.dev at gmail.com> wrote:
> Removes the public symbol _glapi_create_table_from_handle from
> libGL.so.1 on all plattforms except Darwin.
>
typo -> platforms

> Since the symbol is not used on other plattforms it makes sense to
ditto

> build gl_gentable.c only on Darwin.
>
Ideally we'll keep the dispatch as close to identical across all
platforms (i.e. we'll nuke this), although for now this will do.

Out of curiosity is there any noticeable difference in the build times?

...
> XXX If we still want to distribute gl_gentable.c in the release tarball
> we could drop the changes in src/mapi/glapi/gen/Makefile.am
>
Yes please. We want to ship all the generated sources, regardless of
the platform they're used.

...
> index 2da8f7d..25ea44a 100644
> --- a/src/mapi/glapi/gen/Makefile.am
> +++ b/src/mapi/glapi/gen/Makefile.am
> @@ -27,8 +27,11 @@ MESA_GLAPI_OUTPUTS = \
>         $(MESA_GLAPI_DIR)/glapi_mapi_tmp.h \
>         $(MESA_GLAPI_DIR)/glprocs.h \
>         $(MESA_GLAPI_DIR)/glapitemp.h \
> -       $(MESA_GLAPI_DIR)/glapitable.h \
> -       $(MESA_GLAPI_DIR)/glapi_gentable.c
> +       $(MESA_GLAPI_DIR)/glapitable.h
> +
> +if HAVE_APPLEDRI
> +MESA_GLAPI_OUTPUTS += $(MESA_GLAPI_DIR)/glapi_gentable.c
> +endif
>
>  MESA_GLAPI_ASM_OUTPUTS =
>  if HAVE_X86_ASM
> @@ -88,8 +91,11 @@ XORG_GLAPI_DIR = $(XORG_BASE)/glx
>  XORG_GLAPI_OUTPUTS = \
>         $(XORG_GLAPI_DIR)/glprocs.h \
>         $(XORG_GLAPI_DIR)/glapitable.h \
> -       $(XORG_GLAPI_DIR)/dispatch.h \
> +       $(XORG_GLAPI_DIR)/dispatch.h
> +
> +if HAVE_APPLEDRI
>         $(XORG_GLAPI_DIR)/glapi_gentable.c
Erm missing XORG_GLAPI_OUTPUTS += ?

Afaict even with the above makefile changes the file should still be
in the in the tarball. Am I missing something ?

Would be great if Jeremy (or someone else) has the chance to test
this, in case we've missing something.

-Emil