On 05/04/2010 11:17 AM, David Reiss wrote:
> The documentation for AC_ERLANG_SUBST_INSTALL_LIB_DIR states
> """
> its default value is
> @samp{$ERLANG_INSTALL_LIB_DIR/@var{librar...@var{version}}, the value of the
> @code{ERLANG_INSTALL_LIB_DIR} variable being set by the
> @code{AC_ERLANG_SUBST_INSTALL_LIB_DIR} macro.
> """
> (This blames back to 0527671a0606 in 2006.) However, a quick inspection of
> the macro source reveals that the actual default is "LIBDIR/erlang/lib",
> and this was the case even when the documentation was written.
>
> I'd be willing to contribute a patch to either make the docs match the
> implementation or the implementation match the docs, whichever you prefer.
> (I think the latter might be more useful.)Since I'm not an erlang user, I'll leave it to your judgment which is more useful. But for backwards compatibility, I'd prefer a patch along the lines of the former (better to change the docs to match reality, than to change reality and break configure.ac that happened to depend on the old semantics). -- Eric Blake ebl...@redhat.com +1-801-349-2682 Libvirt virtualization library http://libvirt.org
signature.asc
Description: OpenPGP digital signature
