Re: [PATCH FYI] t9700/test.pl: backwards compatibility improvements
- From
Brandon Casey <casey@nrlssc.navy.mil>
- Date
- Sep 15, 2008, 16:20 UTC
- Message-ID
- <ryvGcG_Sg9kftPIp7uWA1JVCd0inSiqB61VnfUuxRUZJg6W7SWhmOQ@cipher.nrlssc.navy.mil>
- In-Reply-To
- <48CE4CFE.1010106@statsbiblioteket.dk>
Tom G. Christensen wrote:
Show 26 quoted lines
> Brandon Casey wrote: >> Some versions of perl complain when 'STDERR' is used as the third >> argument >> in the 3-argument form of open(). Convert to the 2-argument form which is >> described for duping STDERR in my second edition camel book. >> >> The object oriented version of File::Temp is a rather new incarnation it >> seems. The File::Temp man page for v5.8.0 says "(NOT YET IMPLEMENTED)" in >> the 'Objects' section. These can be converted to use >> File::Temp::tempfile(). >> >> Signed-off-by: Brandon Casey <casey@nrlssc.navy.mil> >> > Tested-by: Tom G. Christensen <tgc@statsbiblioteket.dk> > > Without this patch the testsuite will fail with perl 5.8.0: > t9700$ ./test.pl > ok 1 - use Git; > Bareword "STDERR" not allowed while "strict subs" in use at ./test.pl > line 41. > Execution of ./test.pl aborted due to compilation errors. > 1..1 > # Looks like your test died just after 1. > > > Could we please get this patch in?
Junio had asked me to follow up on that patch at a later time, so thanks for prodding.
Here is an alternative which _removes_ the File::Temp requirement.
-brandon