Mercurial > gnulib
changeset 32872:44ff3e757edf
init.sh: simply unset TMPDIR rather than risking env -i
* tests/init.sh (mktempd_): Using env -i is rather harsh, and
although it probably works fine on all Unix-based systems, some
systems (Cygwin?) cannot tolerate a totally cleared environment.
Suggestion from Eric Blake.
2010-04-06 Jim Meyering <meyering@redhat.com>
| author | Jim Meyering <meyering@redhat.com> |
|---|---|
| date | Tue, 06 Apr 2010 19:24:57 +0200 |
| parents | 51dc9555be64 |
| children | 58ed19da9dbd |
| files | ChangeLog tests/init.sh |
| diffstat | 2 files changed, 9 insertions(+), 1 deletions(-) [+] |
line wrap: on
line diff
--- a/ChangeLog Tue Apr 06 17:33:51 2010 +0200 +++ b/ChangeLog Tue Apr 06 19:24:57 2010 +0200 @@ -1,3 +1,11 @@ +2010-04-06 Jim Meyering <meyering@redhat.com> + + init.sh: simply unset TMPDIR rather than risking env -i + * tests/init.sh (mktempd_): Using env -i is rather harsh, and + although it probably works fine on all Unix-based systems, some + systems (Cygwin?) cannot tolerate a totally cleared environment. + Suggestion from Eric Blake. + 2010-04-06 Jim Meyering <meyering@redhat.com> init.sh: portability fix: use env's POSIX-specified -i option not -u
--- a/tests/init.sh Tue Apr 06 17:33:51 2010 +0200 +++ b/tests/init.sh Tue Apr 06 19:24:57 2010 +0200 @@ -306,7 +306,7 @@ fail=0 # First, try to use mktemp. - d=`env -i PATH="$PATH" mktemp -d -t -p "$destdir_" "$template_" 2>/dev/null` \ + d=`unset TMPDIR; mktemp -d -t -p "$destdir_" "$template_" 2>/dev/null` \ || fail=1 # The resulting name must be in the specified directory.