dispatch: setup color before pager for correct console information on windows Before this patch, "hg CMD --pager on" on Windows shows output unintentionally decorated with ANSI color escape sequences, if color mode is "auto". This issue occurs in steps below. 1. dispatch() invokes ui.pager() at detection of "--pager on" 2. stdout of hg process is redirected into stdin of pager process 3. "ui.formatted" = True, because isatty(stdout) is so before (2) 4. color module is loaded for colorization 5. color.w32effects = None, because GetConsoleScreenBufferInfo() fails on stdout redirected at (2) 6. "ansi" color mode is chosen, because of "not w32effects" 7. output is colorized in "ansi" mode because of "ui.formatted" = True Even if "ansi" color mode is chosen, ordinarily redirected stdout makes ui.formatted() return False, and colorization is avoided. But in this issue case, "ui.formatted" = True at (3) forces output to be colorized. For correct console information on win32, it is needed to ensure that color module is loaded before redirection of stdout for pagination. BTW, if any of enabled extensions has "colortable" attribute, this issue is avoided even before this patch, because color module is imported as a part of loading such extension, and extension loading occurs before setting up pager. For example, mq and keyword have "colortable".

     1

from __future__ import absolute_import

     2

import __builtin__

     3

import os

     4

from mercurial import (

     5

    util,

     6

)

     7

     8

def lowerwrap(scope, funcname):

     9

    f = getattr(scope, funcname)

    10

    def wrap(fname, *args, **kwargs):

    11

        d, base = os.path.split(fname)

    12

        try:

    13

            files = os.listdir(d or '.')

    14

        except OSError:

    15

            files = []

    16

        if base in files:

    17

            return f(fname, *args, **kwargs)

    18

        for fn in files:

    19

            if fn.lower() == base.lower():

    20

                return f(os.path.join(d, fn), *args, **kwargs)

    21

        return f(fname, *args, **kwargs)

    22

    scope.__dict__[funcname] = wrap

    23

    24

def normcase(path):

    25

    return path.lower()

    26

    27

os.path.normcase = normcase

    28

    29

for f in 'file open'.split():

    30

    lowerwrap(__builtin__, f)

    31

    32

for f in "chmod chown open lstat stat remove unlink".split():

    33

    lowerwrap(os, f)

    34

    35

for f in "exists lexists".split():

    36

    lowerwrap(os.path, f)

    37

    38

lowerwrap(util, 'posixfile')