From: Alex Riesen Date: Mon, 22 Jan 2007 16:14:56 +0000 (+0100) Subject: Insert ACTIVESTATE_STRING in Git.pm X-Git-Tag: v1.5.0-rc3~87 X-Git-Url: http://git.tremily.us/?a=commitdiff_plain;h=d3b1785f3fe681f6837a6110008af98003f675d3;p=git.git Insert ACTIVESTATE_STRING in Git.pm Also add "git" to the pipe parameters, otherwise it does not work at all, as no git commands are usable out of git context. Signed-off-by: Alex Riesen Signed-off-by: Junio C Hamano --- diff --git a/perl/Git.pm b/perl/Git.pm index 3474ad320..58414e3ec 100644 --- a/perl/Git.pm +++ b/perl/Git.pm @@ -736,7 +736,7 @@ sub _command_common_pipe { _check_valid_cmd($cmd); my $fh; - if ($^O eq '##INSERT_ACTIVESTATE_STRING_HERE##') { + if ($^O eq 'MSWin32') { # ActiveState Perl #defined $opts{STDERR} and # warn 'ignoring STDERR option - running w/ ActiveState'; @@ -809,8 +809,9 @@ sub TIEHANDLE { # FIXME: This is probably horrible idea and the thing will explode # at the moment you give it arguments that require some quoting, # but I have no ActiveState clue... --pasky - my $cmdline = join " ", @params; - my @data = qx{$cmdline}; + # Let's just hope ActiveState Perl does at least the quoting + # correctly. + my @data = qx{git @params}; bless { i => 0, data => \@data }, $class; }