comparison hgext/pager.py @ 30847:e12553cfd0a4 stable

pager: wrap _runcommand() no matter if stdout is redirected

We've made chg utilize the common code path implemented in pager.py (by
815e1cefd082 and 493935e0327a), but the chg server does not always start
with a tty. Because of this, uisetup() of the pager extension could be
skipped on the chg server.

Kudos given to Sean Farley for dogfooding new chg and spotting this problem.

30846:dfc6663f97ca

        os.dup2(stdoutfd, util.stdout.fileno())
        os.dup2(stderrfd, util.stderr.fileno())
        pager.wait()

def uisetup(ui):
    if '--debugger' in sys.argv or not ui.formatted():
        return

    class pagerui(ui.__class__):
        def _runpager(self, pagercmd):
            _runpager(self, pagercmd)

    ui.__class__ = pagerui

        p = ui.config("pager", "pager", encoding.environ.get("PAGER"))
        usepager = False
        always = util.parsebool(options['pager'])
        auto = options['pager'] == 'auto'

        if not p:
            pass
        elif always:
            usepager = True
        elif not auto:
            usepager = False

30847:e12553cfd0a4

        os.dup2(stdoutfd, util.stdout.fileno())
        os.dup2(stderrfd, util.stderr.fileno())
        pager.wait()

def uisetup(ui):
    class pagerui(ui.__class__):
        def _runpager(self, pagercmd):
            _runpager(self, pagercmd)

    ui.__class__ = pagerui

        p = ui.config("pager", "pager", encoding.environ.get("PAGER"))
        usepager = False
        always = util.parsebool(options['pager'])
        auto = options['pager'] == 'auto'

        if not p or '--debugger' in sys.argv or not ui.formatted():
            pass
        elif always:
            usepager = True
        elif not auto:
            usepager = False