[Mesa-dev] [PATCH] build: Share the all-local rule for linking libraries into the build dir

[Matt Turner]

On Wed, Jan 29, 2014 at 11:50 AM, Kristian Høgsberg <krh at bitplanet.net> wrote:
> This consolidates how we link the libraries.  It only works for
> lib_LTLIBRARIES, ie not for custom shared libraries like DRI drivers or
> gallium state trackers.
>
> Signed-off-by: Kristian Høgsberg <krh at bitplanet.net>
> ---
>
> We could do something like this to avoid the problem in the future.
>
> Kristian
>
>  install-lib-links.mk              | 14 ++++++++++++++
>  src/egl/main/Makefile.am          |  7 +------
>  src/gbm/Makefile.am               |  5 +----
>  src/glx/Makefile.am               |  7 +------
>  src/mapi/es1api/Makefile.am       |  8 +-------
>  src/mapi/es2api/Makefile.am       |  8 +-------
>  src/mapi/shared-glapi/Makefile.am |  6 +-----
>  src/mapi/vgapi/Makefile.am        |  8 +-------
>  8 files changed, 21 insertions(+), 42 deletions(-)
>  create mode 100644 install-lib-links.mk
>
> diff --git a/install-lib-links.mk b/install-lib-links.mk
> new file mode 100644
> index 0000000..73d9e14
> --- /dev/null
> +++ b/install-lib-links.mk
> @@ -0,0 +1,14 @@
> +# Provide compatibility with scripts for the old Mesa build system for
> +# a while by putting a link to the driver into /lib of the build tree.
> +
> +all-local : .libs/install-mesa-links
> +
> +.libs/install-mesa-links : $(lib_LTLIBRARIES)
> +       $(MKDIR_P) $(top_builddir)/$(LIB_DIR)
> +       for f in $(lib_LTLIBRARIES:%.la=.libs/%.so*); do        \
> +               if test -h .libs/$$f; then                      \
> +                       cp -d $$f $(top_builddir)/$(LIB_DIR);   \
> +               else                                            \
> +                       ln -f $$f $(top_builddir)/$(LIB_DIR);   \
> +               fi;                                             \
> +       done && touch $@

Oh, that's nice. I like this idea a lot.

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