[Mesa-dev] [PATCH] glsl/build: build builtin_compiler with VISIBILITY_CFLAGS

[Matt Turner]

On Wed, Jul 10, 2013 at 6:14 PM, Chia-I Wu <olvaffe at gmail.com> wrote:
> libglslcore.la and libglcpp.la that are built with builtin_compiler are also
> linked to by drivers not using libdricore.  Since there is no public symbol in
> them, it is better to mark all symbols hidden.
>
> Signed-off-by: Chia-I Wu <olvaffe at gmail.com>
> ---
>  src/glsl/builtin_compiler/Makefile.am | 3 ++-
>  1 file changed, 2 insertions(+), 1 deletion(-)
>
> diff --git a/src/glsl/builtin_compiler/Makefile.am b/src/glsl/builtin_compiler/Makefile.am
> index e11a17f..823d546 100644
> --- a/src/glsl/builtin_compiler/Makefile.am
> +++ b/src/glsl/builtin_compiler/Makefile.am
> @@ -26,7 +26,8 @@ AM_CFLAGS = \
>         -I $(top_srcdir)/src/mesa \
>         -I $(GLSL_SRCDIR) \
>         -I $(GLSL_SRCDIR)/glcpp \
> -       -I $(GLSL_BUILDDIR)
> +       -I $(GLSL_BUILDDIR) \
> +       $(VISIBILITY_CFLAGS)
>
>  if CROSS_COMPILING
>  proxyCC = @CC_FOR_BUILD@
> --
> 1.8.3.1

Seems right.

Reviewed-by: Matt Turner <mattst88 at gmail.com>