Ticket #11898 (closed defect: fixed)
Testing sage/misc/hg.py verbosely starts "less"
| Reported by: | jdemeyer | Owned by: | mvngu |
|---|---|---|---|
| Priority: | blocker | Milestone: | sage-4.7.2 |
| Component: | doctest coverage | Keywords: | mercurial hg pager |
| Cc: | jhpalmieri, kini, kcrisman | Work issues: | |
| Report Upstream: | N/A | Reviewers: | Jeroen Demeyer |
| Authors: | John Palmieri | Merged in: | sage-4.7.2.alpha4 |
| Dependencies: | Stopgaps: |
Attachments
Change History
comment:2 Changed 20 months ago by jhpalmieri
- Status changed from new to needs_review
- Authors set to John Palmieri
That's strange. I don't know the differences between verbose testing and regular that would cause this.
The attached patch fixes the problem for me, but it would also be nice to actually understand what's going on.
comment:3 Changed 20 months ago by jhpalmieri
Another option would be a patch like this (in the pager function):
-
sage/misc/hg.py
diff --git a/sage/misc/hg.py b/sage/misc/hg.py
a b def pager(): 112 112 '--config pager.pager=cat' 113 113 sage: sage.server.support.EMBEDDED_MODE=False 114 114 """ 115 if embedded(): 115 from sage.plot.plot import DOCTEST_MODE 116 if embedded() or DOCTEST_MODE: 116 117 return '--config pager.pager=cat' 117 118 else: 118 119 return '--config pager.pager="LESS=\'R\' less"'
This would require changing the doctests for the pager function, and those doctests wouldn't be as helpful.
Note: See
TracTickets for help on using
tickets.

