autoconf-patches
[Top][All Lists]
Advanced

[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.



reply via email to

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