Code

t9700/test.pl: avoid bareword 'STDERR' in 3-argument open()
authorBrandon Casey <casey@nrlssc.navy.mil>
Mon, 15 Sep 2008 16:23:24 +0000 (11:23 -0500)
committerJunio C Hamano <gitster@pobox.com>
Tue, 16 Sep 2008 06:10:23 +0000 (23:10 -0700)
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.

Signed-off-by: Brandon Casey <casey@nrlssc.navy.mil>
Tested-by: Tom G. Christensen <tgc@statsbiblioteket.dk> on RHEL 3, Perl 5.8.0
Signed-off-by: Junio C Hamano <gitster@pobox.com>
t/t9700/test.pl

index 4d2312548a81762918ac05b9a0014195b08ea532..b47c71989adbfd08e7ee3b37f8457fb378288f67 100755 (executable)
@@ -38,7 +38,7 @@ is($r->get_color("color.test.slot1", "red"), $ansi_green, "get_color");
 # Failure cases for config:
 # Save and restore STDERR; we will probably extract this into a
 # "dies_ok" method and possibly move the STDERR handling to Git.pm.
-open our $tmpstderr, ">&", STDERR or die "cannot save STDERR"; close STDERR;
+open our $tmpstderr, ">&STDERR" or die "cannot save STDERR"; close STDERR;
 eval { $r->config("test.dupstring") };
 ok($@, "config: duplicate entry in scalar context fails");
 eval { $r->config_bool("test.boolother") };