comparison contrib/check-py3-compat.py @ 32212:65cd7e705ff6

policy: eliminate ".pure." from module name only if marked as dual

So we can switch cext/pure modules to new layout one by one.

32211:c48583859e04

import ast
import os
import sys
import traceback

def check_compat_py2(f):
    """Check Python 3 compatibility for a file with Python 2"""
    with open(f, 'rb') as fh:
        content = fh.read()

    # Try to import the module.
    # For now we only support mercurial.* and hgext.* modules because figuring
    # out module paths for things not in a package can be confusing.
    if f.startswith(('hgext/', 'mercurial/')) and not f.endswith('__init__.py'):
        assert f.endswith('.py')
        name = f.replace('/', '.')[:-3].replace('.pure.', '.')
        try:
            importlib.import_module(name)
        except Exception as e:
            exc_type, exc_value, tb = sys.exc_info()
            # We walk the stack and ignore frames from our custom importer,

32212:65cd7e705ff6

import ast
import os
import sys
import traceback

# Modules that have both Python and C implementations.
_dualmodules = (
    'base85.py',
    'bdiff.py',
    'diffhelpers.py',
    'mpatch.py',
    'osutil.py',
    'parsers.py',
)

def check_compat_py2(f):
    """Check Python 3 compatibility for a file with Python 2"""
    with open(f, 'rb') as fh:
        content = fh.read()

    # Try to import the module.
    # For now we only support mercurial.* and hgext.* modules because figuring
    # out module paths for things not in a package can be confusing.
    if f.startswith(('hgext/', 'mercurial/')) and not f.endswith('__init__.py'):
        assert f.endswith('.py')
        name = f.replace('/', '.')[:-3]
        if f.endswith(_dualmodules):
            name = name.replace('.pure.', '.')
        try:
            importlib.import_module(name)
        except Exception as e:
            exc_type, exc_value, tb = sys.exc_info()
            # We walk the stack and ignore frames from our custom importer,