cat: do not start pager if output will be written to file
authorYuya Nishihara <yuya@tcha.org>
Sat, 27 May 2017 18:52:46 +0900
changeset 32541 3b569745af6c
parent 32540 f4cd4c49e302
child 32542 37ea47a50875
cat: do not start pager if output will be written to file
mercurial/commands.py
tests/test-pager.t
--- a/mercurial/commands.py	Sat May 27 18:50:05 2017 +0900
+++ b/mercurial/commands.py	Sat May 27 18:52:46 2017 +0900
@@ -1364,8 +1364,11 @@
     ctx = scmutil.revsingle(repo, opts.get('rev'))
     m = scmutil.match(ctx, (file1,) + pats, opts)
     fntemplate = opts.pop('output', '')
-
-    ui.pager('cat')
+    if cmdutil.isstdiofilename(fntemplate):
+        fntemplate = ''
+
+    if not fntemplate:
+        ui.pager('cat')
     return cmdutil.cat(ui, repo, ctx, m, fntemplate, '', **opts)
 
 @command('^clone',
--- a/tests/test-pager.t	Sat May 27 18:50:05 2017 +0900
+++ b/tests/test-pager.t	Sat May 27 18:52:46 2017 +0900
@@ -1,7 +1,11 @@
   $ cat >> fakepager.py <<EOF
   > import sys
+  > printed = False
   > for line in sys.stdin:
   >     sys.stdout.write('paged! %r\n' % line)
+  >     printed = True
+  > if not printed:
+  >     sys.stdout.write('paged empty output!\n')
   > EOF
 
 Enable ui.formatted because pager won't fire without it, and set up
@@ -281,6 +285,15 @@
    9: a 9
   10: a 10
 
+A command with --output option:
+
+  $ hg cat -r0 a
+  paged! 'a\n'
+  $ hg cat -r0 a --output=-
+  paged! 'a\n'
+  $ hg cat -r0 a --output=out
+  $ rm out
+
 Put annotate in the ignore list for pager:
   $ cat >> $HGRCPATH <<EOF
   > [pager]