Re: Use the pathsep deduced by autoconf

[Gary V. Vaughan]

On Fri, Sep 07, 2001 at 10:02:55AM +0200, Akim Demaille wrote:
> 
> @@ -325,7 +309,7 @@
>    #
>    # So, first we look for a working echo in the user's PATH.
>  
> -  IFS="${IFS=        }"; lt_save_ifs="$IFS"; IFS="${IFS}${PATH_SEPARATOR}"
> +  IFS="${IFS=        }"; lt_save_ifs="$IFS"; IFS="${IFS}$ac_path_separator"
>    for dir in $PATH /usr/ucb; do
>      IFS="$lt_save_ifs"
>      if (test -f $dir/echo || test -f $dir/echo$ac_exeext) &&
> 
> BTW, now Autoconf guarantee a proper IFS, you no longer need to preset
> it. 

Thanks.  I'm checking in a fix...

> But I don't understand why you don't have just
> 
> IFS=$PATH_SEPARATOR
> 
> here.  Why do you keep ` ' etc.?

Looks like a bug to me.  I'm fixing this in the same patch.

  $ ( PATH='C:\Windows;C:\Program Files'; IFS="${IFS};";
  - for dir in $PATH; do echo $dir; done; )
  C:\Windows
  C:\Program
  Files
  $ ( PATH='C:\Windows;C:\Program Files'; IFS=";";
  - for dir in $PATH; do echo $dir; done; )
  C:\Windows
  C:\Program Files

Cheers,
        Gary.
-- 
  ())_. Gary V. Vaughan     gary@(oranda.demon.co.uk|gnu.org)
  ( '/  Research Scientist  http://www.oranda.demon.co.uk       ,_())____
  / )=  GNU Hacker          http://www.gnu.org/software/libtool  \'      `&
`(_~)_  Tech' Author        http://sources.redhat.com/autobook   =`---d__/