On 09/06/2010 12:07 AM, Ralf Wildenhues wrote:
* doc/autoconf.texi: Merge variable index `vr' into concept
index using syncodeindex.
* doc/autoconf.texi (Configuration Actions, Generic Programs)
(Generic Functions, Initialization Macros, Writing Testsuites):
Add index entries for documented shell variables used during
in configure and testsuite scripts.  Fix entry for `tmp'.


  @table @code
  @item srcdir
+...@vrindex srcdir

Why @vrindex instead of @ovindex?  We already document:

@c Define an output variable index.
@defcodeindex ov

for all variables that are set during configure/autotest for use by the remainder of the script.

  @item tmp
+...@vrindex tmp
  The name of a temporary directory within the build tree, which you
  can use if you need to create additional temporary files.  The
  directory is cleaned up when @command{config.status} is done or

In fact, I just added @ovindex tmp to AS_TMPDIR recently.


@@ -4316,7 +4324,7 @@ Generic Programs
    @ovar{action-if-not-found}, @dvar{path, $PATH})
  @acindex{PATH_PROGS_FEATURE_CHECK}
  @caindex pa...@var{variable}
-...@evindex ac_pa...@var{variable}
+...@vrindex ac_pa...@var{variable}

This hunk may conflict with the resolution for the issue raised resolution on your previous patch.

@@ -13706,7 +13715,7 @@ Initialization Macros
  @defmac AS_TMPDIR (@var{prefix}, @dvar{dir, $...@{tmpdir:=/t...@}})
  @asindex{TMPDIR}
  @evindex TMPDIR
-...@ovindex tmp
+...@vrindex tmp
  Create, as safely as possible, a temporary sub-directory within
  @var{dir} with a name starting with @var{prefix}.  @var{prefix} should
  be 2-4 characters, to make it slightly easier to identify the owner of

Oh, I see - you're trying to make a distinction between @ovindex and @vrindex. Can we document that distinction both in the commit message and in the comments at the top of autoconf.texi that describe all the various indices?

I think we'll need another attempt at this patch before pushing anything.

--
Eric Blake   [email protected]    +1-801-349-2682
Libvirt virtualization library http://libvirt.org

Reply via email to