[Mesa-dev] [PATCH] i965: Link i965_dri.so with C++ linker.

[Matt Turner]

On Thu, Feb 28, 2013 at 6:21 PM, Frank Henigman <fjhenigman at google.com> wrote:
> Force C++ linking of i965_dri.so by adding a dummy C++ source file.
> ---
>  src/mesa/drivers/dri/i965/Makefile.am | 1 +
>  1 file changed, 1 insertion(+)
>
> diff --git a/src/mesa/drivers/dri/i965/Makefile.am b/src/mesa/drivers/dri/i965/Makefile.am
> index dc140df..77670ef 100644
> --- a/src/mesa/drivers/dri/i965/Makefile.am
> +++ b/src/mesa/drivers/dri/i965/Makefile.am
> @@ -62,6 +62,7 @@ TEST_LIBS = \
>         ../common/libdri_test_stubs.la
>
>  i965_dri_la_SOURCES =
> +nodist_EXTRA_i965_dri_la_SOURCES = dummy2.cpp
>  i965_dri_la_LIBADD = $(COMMON_LIBS)
>  i965_dri_la_LDFLAGS = -module -avoid-version -shared
>
> --
> 1.8.1.3

Oh god build systems are so broken. Seriously, why isn't automake able
to recognize that linking a convenience library with C++ code in it
into another library that it needs to do it with the C++ linker?
Presumably you were seeing some undefined references. Which ones?

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