[Top][All Lists]

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: AC_PREFIX_PATH, AS_BASENAME fixes and a new test!

From: Akim Demaille
Subject: Re: AC_PREFIX_PATH, AS_BASENAME fixes and a new test!
Date: 17 Sep 2001 16:12:31 +0200
User-agent: Gnus/5.0808 (Gnus v5.8.8) XEmacs/21.4 (Artificial Intelligence)

| Hope you find these useful, there are actually five patches here, but I put
| two of them into the same file. I will describe the patch attachments and
| the bugs fixed or features added.

All this seems good to me, thanks! Please send it as a single patch
with a ChangeLog entry.

| autoconf.fix_prefix.path.gz
|   -- In autoconf 2.13, and in the info pages, it states that AC_PREFIX_PATH
| will return the prefix of a binary. the examples given are using gcc located
| in '/usr/local/bin/gcc' should return '/usr/local'. autoconf 2.13 does do
| this! the current CVS does not! it returns '/usr/local/bin'. This patch
| corrects that behaviour.


| Finally, maybe this should be discussed elsewhere, but there should be a fix
| to the info documentation. In the section on unreliable commands it
| describes:
| `dirname'
|      Not all hosts have `dirname', but it is reasonably easy to emulate,
| e.g.:
|           dir=`expr "x$file" : 'x\(.*\)/[^/]*' \|
|                     '.'      : '.'
| Perhaps this should refer to the AS_DIRNAME macro, as it is actually
| guaranteed to work, since it has the sed fallback...

This might no be the proper place to document AS_DIRNAME, but it it OK
to document the expr and sed based alternative in their full length.

| Maybe there should also be a section added on the basename command? If
| people agree, then I will make up a patch for the english documentation.
| Please let me know.

That would be perfect!

Thanks _a lot_.

reply via email to

[Prev in Thread] Current Thread [Next in Thread]