autoconf-patches
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [PATCH] autoconf: Create LIBOBJ_DIR in $top_builddir.


From: Nick Bowler
Subject: Re: [PATCH] autoconf: Create LIBOBJ_DIR in $top_builddir.
Date: Thu, 20 Jul 2017 09:36:53 -0400

Hello,

On 2017-07-20, Michael Haubenwallner
<address@hidden> wrote:
> * lib/autoconf/general.m4 (AC_CONFIG_LIBOBJ_DIR): Have config.status
> create $top_builddir/LIBOBJ_DIR by INIT-CMDS, to support out of source
> builds even when nothing else does create $top_builddir/LIBOBJ_DIR.
> Based on discussion in https://github.com/westes/flex/issues/244.
> ---
>  lib/autoconf/general.m4 | 4 +++-
>  1 file changed, 3 insertions(+), 1 deletion(-)
>
> diff --git a/lib/autoconf/general.m4 b/lib/autoconf/general.m4
> index f783404..4b152df 100644
> --- a/lib/autoconf/general.m4
> +++ b/lib/autoconf/general.m4
> @@ -3014,8 +3014,10 @@ AC_DEFUN([AC_CHECK_DECLS_ONCE],
>  # AC_CONFIG_LIBOBJ_DIR(DIRNAME)
>  # -----------------------------
>  # Announce LIBOBJ replacement files are in $top_srcdir/DIRNAME.
> +# Creating $top_builddir/DIRNAME is for out of source builds.
>  AC_DEFUN_ONCE([AC_CONFIG_LIBOBJ_DIR],
> -[m4_divert_text([DEFAULTS], [ac_config_libobj_dir=$1])])
> +[m4_divert_text([DEFAULTS], [ac_config_libobj_dir=$1])dnl
> +AC_CONFIG_COMMANDS(,,[\$MKDIR_P "$ac_config_libobj_dir"])])

This is probably a good idea in principle, but the fix should probably
be done in Automake, since the actual failure is really an Automake
issue.

I suggest that Automake should simply create all the build directories
it normally would regardless of whether or not dependency tracking is
enabled.

Cheers,
  Nick



reply via email to

[Prev in Thread] Current Thread [Next in Thread]