On Fri, Sep 19, 2014 at 4:01 PM, Emil Velikov <emil.l.veli...@gmail.com> wrote:
> Ensure that the object is build in the target folder, as automake 2.0
> will mandate subdir-objects. Pointed out by automake 1.14.
>
> Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=69874
> Signed-off-by: Emil Velikov <emil.l.veli...@gmail.com>
> ---
>  src/gallium/targets/omx/.gitignore  |  1 +
>  src/gallium/targets/omx/Makefile.am | 10 ++++++++--
>  2 files changed, 9 insertions(+), 2 deletions(-)
>  create mode 100644 src/gallium/targets/omx/.gitignore
>
> diff --git a/src/gallium/targets/omx/.gitignore 
> b/src/gallium/targets/omx/.gitignore
> new file mode 100644
> index 0000000..4fd1800
> --- /dev/null
> +++ b/src/gallium/targets/omx/.gitignore
> @@ -0,0 +1 @@
> +vl_winsys_dri.c
> diff --git a/src/gallium/targets/omx/Makefile.am 
> b/src/gallium/targets/omx/Makefile.am
> index 4045548..f41719f 100644
> --- a/src/gallium/targets/omx/Makefile.am
> +++ b/src/gallium/targets/omx/Makefile.am
> @@ -7,8 +7,7 @@ omxdir = $(OMX_LIB_INSTALL_DIR)
>  omx_LTLIBRARIES = libomx_mesa.la
>
>  nodist_EXTRA_libomx_mesa_la_SOURCES = dummy.cpp
> -libomx_mesa_la_SOURCES = \
> -       $(top_srcdir)/src/gallium/auxiliary/vl/vl_winsys_dri.c
> +libomx_mesa_la_SOURCES = vl_winsys_dri.c
>
>  libomx_mesa_la_LDFLAGS = \
>         -shared \
> @@ -30,6 +29,13 @@ libomx_mesa_la_LIBADD = \
>         $(OMX_LIBS) \
>         $(GALLIUM_COMMON_LIB_DEPS)
>
> +BUILT_SOURCES = vl_winsys_dri.c
> +CLEANFILES = vl_winsys_dri.c
> +
> +vl_winsys_dri.c:
> +       $(AM_V_GEN)$(LN_S) 
> $(top_srcdir)/src/gallium/auxiliary/vl/vl_winsys_dri.c

This file gets built by omx, xvmc, and vdpau, but is it actually built
with different CPPFLAGS or something? That is, can't we actually just
build it once in its subdirectory?

I don't see any meaningful preprocessor checks in the source file that
make me think it can't.
_______________________________________________
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/mesa-dev

Reply via email to