Skip to content

Commit

Permalink
Ensure builtin modules are from typeshed sooner
Browse files Browse the repository at this point in the history
It should work now with custom-typeshed-dir.

Fixes python#1876
  • Loading branch information
Konstantin Ignatov committed Jul 17, 2022
1 parent ca6357e commit 087db77
Show file tree
Hide file tree
Showing 7 changed files with 65 additions and 14 deletions.
1 change: 1 addition & 0 deletions build-requirements.txt
Original file line number Diff line number Diff line change
@@ -1,2 +1,3 @@
-r mypy-requirements.txt
types-setuptools
types-typed-ast>=1.5.0,<1.6.0
1 change: 1 addition & 0 deletions mypy-requirements.txt
Original file line number Diff line number Diff line change
@@ -1,4 +1,5 @@
typing_extensions>=3.10
mypy_extensions>=0.4.3
import_resources; python_version<'3.7'
typed_ast>=1.4.0,<2; python_version<'3.8'
tomli>=1.1.0; python_version<'3.11'
34 changes: 25 additions & 9 deletions mypy/build.py
Original file line number Diff line number Diff line change
Expand Up @@ -605,6 +605,31 @@ def __init__(self, data_dir: str,
self.fscache = fscache
self.find_module_cache = FindModuleCache(self.search_paths, self.fscache, self.options,
source_set=self.source_set)
for module in CORE_BUILTIN_MODULES:
if options.use_builtins_fixtures:
continue
if module == "_importlib_modulespec":
continue
path = self.find_module_cache.find_module(module)
if not isinstance(path, str):
raise CompileError([
f"Failed to find builtin module {module}, perhaps typeshed is broken?",
])
if is_typeshed_file(path):
continue
if is_stub_package_file(path):
continue
if options.custom_typeshed_dir is not None:
# Check if module lives under custom_typeshed_dir subtree
custom_typeshed_dir = os.path.abspath(options.custom_typeshed_dir)
if os.path.commonpath((path, custom_typeshed_dir)) == custom_typeshed_dir:
continue

raise CompileError([
f'mypy: "{os.path.relpath(path)}" shadows library module "{module}"',
f'note: A user-defined top-level module with name "{module}" is not supported'
])

self.metastore = create_metastore(options)

# a mapping from source files to their corresponding shadow files
Expand Down Expand Up @@ -2459,15 +2484,6 @@ def find_module_and_diagnose(manager: BuildManager,
if is_sub_path(result, dir):
# Silence errors in site-package dirs and typeshed
follow_imports = 'silent'
if (id in CORE_BUILTIN_MODULES
and not is_typeshed_file(result)
and not is_stub_package_file(result)
and not options.use_builtins_fixtures
and not options.custom_typeshed_dir):
raise CompileError([
f'mypy: "{os.path.relpath(result)}" shadows library module "{id}"',
f'note: A user-defined top-level module with name "{id}" is not supported'
])
return (result, follow_imports)
else:
# Could not find a module. Typically the reason is a
Expand Down
1 change: 1 addition & 0 deletions mypy/test/testgraph.py
Original file line number Diff line number Diff line change
Expand Up @@ -38,6 +38,7 @@ def test_scc(self) -> None:
def _make_manager(self) -> BuildManager:
errors = Errors()
options = Options()
options.use_builtins_fixtures = True
fscache = FileSystemCache()
search_paths = SearchPaths((), (), (), ())
manager = BuildManager(
Expand Down
18 changes: 15 additions & 3 deletions mypy/util.py
Original file line number Diff line number Diff line change
Expand Up @@ -9,6 +9,10 @@
import io
import shutil
import time
try:
from importlib import resources as import_resources # type: ignore[attr-defined]
except ImportError: # <python3.7
import import_resources # type: ignore[import]

from typing import (
TypeVar, List, Tuple, Optional, Dict, Sequence, Iterable, Container, IO, Callable, Union, Sized
Expand All @@ -22,6 +26,12 @@
except ImportError:
CURSES_ENABLED = False

with import_resources.path(
'mypy' if __package__ is None else __package__, # mypy-c doesn't support __package__
".",
) as _resource:
TYPESHED_DIR: Final = str(_resource / "typeshed")

T = TypeVar('T')

ENCODING_RE: Final = re.compile(br"([ \t\v]*#.*(\r\n?|\n))??[ \t\v]*#.*coding[:=][ \t]*([-\w.]+)")
Expand Down Expand Up @@ -745,16 +755,18 @@ def format_error(


def is_typeshed_file(file: str) -> bool:
# gross, but no other clear way to tell
return 'typeshed' in os.path.abspath(file).split(os.sep)
try:
return os.path.commonpath((TYPESHED_DIR, os.path.abspath(file))) == TYPESHED_DIR
except ValueError: # Different drives on Windows
return False


def is_stub_package_file(file: str) -> bool:
# Use hacky heuristics to check whether file is part of a PEP 561 stub package.
if not file.endswith('.pyi'):
return False
return any(component.endswith('-stubs')
for component in os.path.abspath(file).split(os.sep))
for component in os.path.split(os.path.abspath(file)))


def unnamed_function(name: Optional[str]) -> bool:
Expand Down
4 changes: 2 additions & 2 deletions mypyc/build.py
Original file line number Diff line number Diff line change
Expand Up @@ -47,7 +47,7 @@

try:
# Import setuptools so that it monkey-patch overrides distutils
import setuptools # type: ignore # noqa
import setuptools # noqa
except ImportError:
if sys.version_info >= (3, 12):
# Raise on Python 3.12, since distutils will go away forever
Expand All @@ -63,7 +63,7 @@ def get_extension() -> Type['Extension']:
if not use_setuptools:
from distutils.core import Extension
else:
from setuptools import Extension # type: ignore # noqa
from setuptools import Extension # noqa

return Extension

Expand Down
20 changes: 20 additions & 0 deletions test-data/unit/cmdline.test
Original file line number Diff line number Diff line change
Expand Up @@ -1425,3 +1425,23 @@ b\.c \d+
# cmd: mypy --enable-incomplete-features a.py
[file a.py]
pass

[case testShadowTypingModuleEarlyLoad]
# cmd: mypy dir
[file dir/__init__.py]
from typing import Union

def foo(a: Union[int, str]) -> str:
return str
[file typing.py]
import sys
import os
del sys.modules["typing"]
path = sys.path
sys.path.remove(os.getcwd())
from typing import *
sys.path = path
[out]
mypy: "typing.py" shadows library module "typing"
note: A user-defined top-level module with name "typing" is not supported
== Return code: 2

0 comments on commit 087db77

Please sign in to comment.