[Mesa-dev] [PATCH v3] glapi: Build glapi_gentable.c only on Darwin

[Andreas Boll]

Thanks for reviewing!

Oh, I was motivated to do this patch to remove an unneeded symbol on
linux as we started tracking libGL's exported symbols in Debian.
As a side effect it accelerates the build a bit and reduces the size
of libGL significantly.

>From looking at these threads [1][2] I found out that Arlie's patch
was reviewed but not pushed due to missing testing on OS X.
His patch would still make sense for OS X. I'll ping Jeremy on that thread.

Anyway I'll add the reported-by line.

Thanks,
Andreas

[1] https://patchwork.freedesktop.org/patch/59617/
[2] https://patchwork.freedesktop.org/patch/59812/

2016-01-13 20:50 GMT+01:00 Matt Turner <mattst88 at gmail.com>:
> glxgears still works for me, and libGL goes from 4.2M to 3.3M.
>
> Reviewed-by: Matt Turner <mattst88 at gmail.com>
>
> We should also include some mention of Arlie's contribution, since he
> identified this and sent the initial patch:
>
> Reported-by: Arlie Davis <arlied at google.com>