Re: Incorrect documentation for AC_ERLANG_SUBST_INSTALL_LIB_DIR

[Eric Blake]

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/@address@hidden, 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   address@hidden    +1-801-349-2682
Libvirt virtualization library http://libvirt.org

signature.asc
Description: OpenPGP digital signature