Re: 284-gary-report-macro-files-for-aclocal.diff

[Ralf Wildenhues]

Hi Gary,

* Gary V. Vaughan wrote on Fri, Sep 30, 2005 at 12:23:03PM CEST:
> Okay to commit?

Observe major nit below, then commit, please!

By the way, I believe Patrick meant a generic command line switch to
list the necessary macro files.

Cheers, and thanks,
Ralf


> from  Gary V. Vaughan  <address@hidden>
>       * libtoolize.m4sh: Copying just libtool.m4 for hand maintained
>       aclocal.m4 doesn't work.  List all required files in that case.
> 
> Index: libtool--devo--1.0/libtoolize.m4sh
> ===================================================================
> --- libtool--devo--1.0.orig/libtoolize.m4sh
> +++ libtool--devo--1.0/libtoolize.m4sh
> @@ -996,7 +996,13 @@ func_nonemptydir_p ()
>      func_copy_some_files "$pkgmacro_files" "$aclocaldir" \
>        "$m4dir" func_serial_update
>    else
> -      $opt_quiet || func_echo "You should add the contents of 
> \`$aclocaldir/libtool.m4' to \`aclocal.m4'."
> +      $opt_quiet || {
> +          func_echo "You should add the contents of the following files to 
> \`aclocal.m4':"
> +       for need in libtool.m4 ltoptions.m4 ltversion.m4 ltsugar.m4; do
> +         func_echo "\`$aclocaldir/libtool.m4'"

  s,libtool\.m4,$need,

> +       done
> +       $seen_ltdl && func_echo "\`$aclocaldir/ltdl.m4'"
> +      }
>    fi
>  
>    # Suggest modern idioms for storing autoconf macros: