[gmx-developers] problem with Makefile.am

[Mark Abraham]

Hi,

Christoph Jungan's Feburary commit did

> commit b7aaec78121ba48e5ba03071ad4ba4d95a4649d9
> Author: Christoph Junghans <junghans at mpip-mainz.mpg.de>
> Date:   Thu Feb 18 12:43:35 2010 +0100
>
>     Dep issues with libgmxpreprocess (#392, #395).
>
>     libgmxpreprocess now depends on libmd, and is built/installed with
>     make mdrun/install-mdrun.
>
> diff --git a/Makefile.am b/Makefile.am
> index 52a70cb..f94d4db 100644
> --- a/Makefile.am
> +++ b/Makefile.am
> @@ -35,11 +35,13 @@ install-exec-hook:
>  mdrun:
>         (cd $(top_builddir)/src/gmxlib && $(MAKE) ; exit 0)
>         (cd $(top_builddir)/src/mdlib && $(MAKE) ; exit 0)
> +       (cd $(top_builddir)/src/kernel && $(MAKE) lib_LTLIBRARIES ; exit 0)
>         (cd $(top_builddir)/src/kernel && $(MAKE) mdrun ; exit 0)
>
>  install-mdrun:
>         (cd $(top_builddir)/src/gmxlib && $(MAKE) install ; exit 0)
>         (cd $(top_builddir)/src/mdlib && $(MAKE) install ; exit 0)
> +       (cd $(top_builddir)/src/kernel && $(MAKE) install-libLTLIBRARIES ; exit 0)
>         (cd $(top_builddir)/src/kernel && $(MAKE) install-mdrun ; exit 0)
>
>  fahcore:
>

In "make mdrun" lib_LTLIBRARIES is a variable, not a make target in 
src/kernel/Makefile, so I think either the relevant line should be

        (cd $(top_builddir)/src/kernel && $(MAKE) $(lib_LTLIBRARIES) ; 
exit 0)

or there's a make target missing.

The "install-libLTLIBRARIES" in "make install-mdrun" is ok, however.

Mark