[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: tvh-2-pathsep-cdpath.diff
From: |
Akim Demaille |
Subject: |
Re: tvh-2-pathsep-cdpath.diff |
Date: |
17 Sep 2001 11:16:29 +0200 |
User-agent: |
Gnus/5.0808 (Gnus v5.8.8) XEmacs/21.4 (Artificial Intelligence) |
| Akim, look what you did: because you use APatche as alias
| for this list, M$ Outlook automagically uses it too :-)
:)
| -A simple colon is enough except for @code{zsh}, which prefers a leading
| -dot:
| +A simple path separator is enough except for @code{zsh}, which prefers a
| +leading dot:
|
| @example
| -zsh-3.1.6 % mkdir foo && (CDPATH=: cd foo)
| +zsh-3.1.6 % mkdir foo && (address@hidden@} cd foo)
| /tmp/foo
| -zsh-3.1.6 % (CDPATH=:. cd foo)
| +zsh-3.1.6 % (address@hidden@}. cd foo)
| /tmp/foo
| -zsh-3.1.6 % (CDPATH=.: cd foo)
| +zsh-3.1.6 % (address@hidden@} cd foo)
| zsh-3.1.6 %
| @end example
I'm not too much in favor of replacing `:' with PATH_SEPARATOR in the
example and discussions about CDPATH, since it obfuscates what we are
trying to teach, mentioning that using PATH_SEPARATOR is better style
seems better to me. But I leave the final decision to you.
| +If it is not set, @command{autoconf} will detect the appropriate path
| +separator for the build system and set the @code{PATH_SEPARATOR} output
| +variable accordingly.
You might intended to have a empty line here. And referring to
autoconf here is somewhat misleading: configure does.
| +On DJGPP systems, the @code{PATH_SEPARATOR} environment variable can be
| +set to either @samp{:} or @samp{;} to control the path separator
| address@hidden uses to set up certain environment variables (such as
| address@hidden). Since this only works inside bash, you want autoconf to
| +detect the regular @sc{dos} path separator (@samp{;}), so it can be safely
| +substituted in files that may not support @samp{;} as path separator. So
| either unset this variable or set it to @samp{;}.
Please, keep the typography consistent: @command{autoconf},
@command{bash} etc. You may chose between Autoconf the package here,
or @command{configure} to be specific. I prefer the former, but chose
what you want.
| Index: m4/auxdir.m4
| ===================================================================
| RCS file: /cvs/autoconf/m4/auxdir.m4,v
| retrieving revision 1.1
| diff -u -r1.1 auxdir.m4
| --- m4/auxdir.m4 2001/08/27 07:12:17 1.1
| +++ m4/auxdir.m4 2001/09/16 09:33:19
| @@ -40,5 +40,5 @@
|
| AC_DEFUN([AM_AUX_DIR_EXPAND], [
| # expand $ac_aux_dir to an absolute path
| -am_aux_dir=`CDPATH=:; cd $ac_aux_dir && pwd`
| +am_aux_dir=`cd $ac_aux_dir && pwd`
| ])
If you do this, please be sure to AC_PREREQ(2.50), since 2.13 does not
enforce a proper CDPATH. So this code, and the other m4/*.m4 files,
depends on at least 2.50.
And you said, they come from Automake, so don't forget to fix them in
CVS Automake, then update the Autoconf copies.