Re: mktemp: not found

2011-12-05 Thread Eric Blake
On 12/04/2011 12:57 PM, Jim Meyering wrote: >> - d=`unset TMPDIR; mktemp -d -t -p "$destdir_" "$template_" 2>/dev/null` \ >> + d=`unset TMPDIR; (mktemp -d -t -p "$destdir_" "$template_") 2>/dev/null` \ > > Thanks. That would avoid the spurious output. > How about using { ...; } instead? > >

Re: mktemp: not found

2011-12-04 Thread Bruno Haible
Jim Meyering wrote: > How about using { ...; } instead? > > d=`unset TMPDIR; { mktemp -d -t -p "$destdir_" "$template_"; } 2>/dev/null` > \ > > That should do the same without the cost of a sub-shell. Yes, this works too. Applied, with this ChangeLog entry: 2011-12-04 Bruno Haible

Re: mktemp: not found

2011-12-04 Thread Bruce Korb
On 12/04/11 11:57, Jim Meyering wrote: d=`unset TMPDIR; { mktemp -d -t -p "$destdir_" "$template_"; } 2>/dev/null` \ That should do the same without the cost of a sub-shell. Or: d=` exec 2>/dev/null unset TMPDIR mktemp -d -t -p "$destdir_" "$template_" ` as we discovered re

Re: mktemp: not found

2011-12-04 Thread Jim Meyering
Bruno Haible wrote: > On Minix 3.1.8, I get this spurious output from the test-verify.sh test: > > mktemp: not found > PASS: test-verify.sh > > This patch fixes it. OK to apply? > > 2011-12-04 Bruno Haible > > tests: Avoid spurious error message on platforms without mktemp program. >