bug-coreutils
[Top][All Lists]
Advanced

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

Re: env+nice, one bug fix, two test corrections


From: Eric Blake
Subject: Re: env+nice, one bug fix, two test corrections
Date: Mon, 26 Oct 2009 07:06:57 -0600
User-agent: Mozilla/5.0 (Windows; U; Windows NT 5.1; en-US; rv:1.8.1.23) Gecko/20090812 Thunderbird/2.0.0.23 Mnenhy/0.7.6.666

-----BEGIN PGP SIGNED MESSAGE-----
Hash: SHA1

According to Jim Meyering on 10/26/2009 3:04 AM:
>  # Use -- to end arguments.
> -cat <<EOF >./-i || framework_failure
> -#!/bin/sh
> -echo pass
> -EOF
> -chmod +x ./-i || framework_failure
> -case `env -i PATH="$PATH" echo good` in
> +ln -s "$abs_top_builddir/src/echo" ./-i || framework_failure
> +case `PATH="$PATH:" env -i echo good` in
>    good) ;;
>    *) fail=1 ;;
>  esac

This is still failing for me on cygwin:

> + ln -s /home/eblake/coreutils/src/echo ./-i
> + case `PATH="$PATH:" env -i echo good` in
> ++ 
> PATH='/home/eblake/coreutils/src:/usr/local/bin:/usr/bin:/bin:/usr/X11R6/bin:'
> ++ env -i echo good
> env: echo: No such file or directory
> + fail=1

In other words, cygwin's execvp can't find /bin/echo once -i cleaned the
environment.

> +case `env -i -- PATH=. -i no-echo` in
> +  no-echo) ;;
>    *) fail=1 ;;
>  esac

In a similar vein, cygwin REQUIRES that path include /bin, in order to
find cygwin1.dll; restricting to just dot is too tight:

> + case `env -i -- PATH=. -i no-echo` in
> ++ env -i -- PATH=. -i no-echo
> + fail=1

Also, did you really mean to include -i twice in that line?

I'll play with this some more and propose a patch later.

- --
Don't work too hard, make some time for fun as well!

Eric Blake             address@hidden
-----BEGIN PGP SIGNATURE-----
Version: GnuPG v1.4.9 (Cygwin)
Comment: Public key at home.comcast.net/~ericblake/eblake.gpg
Comment: Using GnuPG with Mozilla - http://enigmail.mozdev.org/

iEYEARECAAYFAkrlnvEACgkQ84KuGfSFAYBg2QCgvZtDWoZUF4B/29rTJf0VWKkg
41gAn3e31eT3Skkice3VA7XnIPJgeUaB
=z7rH
-----END PGP SIGNATURE-----




reply via email to

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