Even though config_pager() unset the $pager variable, we were
blindly calling exec() on it through run_pager().
Noticed-by: Chris Moore <christopher.ian.moore@xxxxxxxxx>
Signed-off-by: Eric Wong <normalperson@xxxxxxxx>
---
Chris Moore <christopher.ian.moore@xxxxxxxxx> wrote:
> I run my git commands inside an Emacs *shell* buffer, so I don't want
> it running 'less' all the time. I export GIT_PAGER=cat to stop it.
>
> I just tried a "git-svn log" and it failed:
>
> $ GIT_PAGER=cat git-svn log
> Use of uninitialized value in exec at /usr/bin/git-svn line 3451.
> Use of uninitialized value in concatenation (.) or string at
> /usr/bin/git-svn line 3451.
> Can't run pager: Illegal seek ()
>
> I don't get a shell prompt back. It just hangs.
>
> This is on ubuntu feisty with the ubuntu-backport repositories enabled:
>
> ii git-core 1.5.2.5-2~feisty1
> ii git-svn 1.5.2.5-2~feisty1
git-svn.perl | 2 +-
1 files changed, 1 insertions(+), 1 deletions(-)
diff --git a/git-svn.perl b/git-svn.perl
index 288d32c..484b057 100755
--- a/git-svn.perl
+++ b/git-svn.perl
@@ -3578,7 +3578,7 @@ sub config_pager {
}
sub run_pager {
- return unless -t *STDOUT;
+ return unless -t *STDOUT && defined $pager;
pipe my $rfd, my $wfd or return;
defined(my $pid = fork) or ::fatal "Can't fork: $!\n";
if (!$pid) {
--
Eric Wong
-
To unsubscribe from this list: send the line "unsubscribe git" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
