--- a/tests/test-pager.t Fri Mar 17 19:12:22 2017 +0530
+++ b/tests/test-pager.t Wed Mar 15 20:33:47 2017 -0400
@@ -119,6 +119,33 @@
paged! 'summary: modify a 8\n'
paged! '\n'
+An invalid pager command name is reported sensibly if we don't have to
+use shell=True in the subprocess call:
+ $ hg log --limit 3 --config pager.pager=this-command-better-never-exist
+ missing pager command 'this-command-better-never-exist', skipping pager
+ \x1b[0;33mchangeset: 10:46106edeeb38\x1b[0m (esc)
+ tag: tip
+ user: test
+ date: Thu Jan 01 00:00:00 1970 +0000
+ summary: modify a 10
+
+ \x1b[0;33mchangeset: 9:6dd8ea7dd621\x1b[0m (esc)
+ user: test
+ date: Thu Jan 01 00:00:00 1970 +0000
+ summary: modify a 9
+
+ \x1b[0;33mchangeset: 8:cff05a6312fe\x1b[0m (esc)
+ user: test
+ date: Thu Jan 01 00:00:00 1970 +0000
+ summary: modify a 8
+
+
+A complicated pager command gets worse behavior. Bonus points if you can
+improve this.
+ $ hg log --limit 3 \
+ > --config pager.pager='this-command-better-never-exist --seriously' \
+ > 2>/dev/null || true
+
Pager works with shell aliases.
$ cat >> $HGRCPATH <<EOF