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