Re: [PATCH v2] aclocal: add --aclocal-path option to override $ACLOCAL_PATH

Karl Berry <[email protected]>
Newsgroups gmane.comp.sysutils.automake.patches
Message-ID <[email protected]>
+      --aclocal-path=PATHS  colon-separated paths to search for third-party
+                              local files

Maybe "colon-separated list of directories".
Or "colon-separated path".

    +@item --aclocal-path=@var{dir}

@var{dir} -> @var{path}

    +@opindex --aclocal-path
    +Look for local third-party macro files (and the special @file{dirlist}
    +file) in @var{dir} instead of in the installation directory.  This is

in @var{dir} -> along @var{path} (a colon-separated list of directories)

    +typically used for building against local sysroots.

Can we link to somewhere that explains what this means?
Or at least have a parenthetical sentence of explanation?

    +@vindex ACLOCAL_PATH
    +The environment variable @env{ACLOCAL_PATH} provides another way to
    +set the directory containing local third-party macro files.  

s/directory/search path for directories/

    ... However
    +@option{--aclocal-path} takes precedence over it.

It's not really "However". Also, it needs to be clear that if
--aclocal-path is specified. How about:

 This variable is ignored if the @{--ac-local-path} option is specified.

    -@subsubheading Modifying the Macro Search Path: @file{ACLOCAL_PATH}
    +@subsubheading Modifying the Macro Search Path: @option{--aclocal-path}

It seems you've sometimes replaced ACLOCAL_PATH with --aclocal-path.
But ACLOCAL_PATH still exists. So it seems like both should be
mentioned.  You do that in some places, but didn't seem like all.

Thanks,
Karl
lmpx.com only provides a reader for public news (NNTP) servers. It is not affiliated with the servers or forums shown here and is not responsible for the content of articles, which is written by their respective authors.