Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Skip compiling mypyc/__main__.py #11862

Merged
merged 2 commits into from Dec 29, 2021

Conversation

mr-c
Copy link
Contributor

@mr-c mr-c commented Dec 29, 2021

Description

Skip compiling mypyc/__main__.py to fix mypyc/mypyc#912

Test Plan

With this patch, the tests testCompileMypyc and testErrorOutput pass during the building of the mypy 0.930 binary packages for Debian.

As for CI testing, in Debian we don't build via python3 setup.py --use-mypyc build_ext --inplace we use MYPY_USE_MYPYC=1 python3 setup.py build and then copy the results into a temporary directory, where we then run the tests.

setup.py Outdated Show resolved Hide resolved
@hauntsaninja
Copy link
Collaborator

Thanks, updated to add a comment
CI should catch this now, see #11865 (comment)

@hauntsaninja hauntsaninja merged commit fe3b625 into python:master Dec 29, 2021
@mr-c mr-c deleted the skip_compiling_mypyc_main branch December 29, 2021 19:11
JukkaL pushed a commit that referenced this pull request Jan 5, 2022
Fixes mypyc/mypyc#912

Co-authored-by: Shantanu <12621235+hauntsaninja@users.noreply.github.com>
tushar-deepsource pushed a commit to DeepSourceCorp/mypy that referenced this pull request Jan 20, 2022
Fixes mypyc/mypyc#912

Co-authored-by: Shantanu <12621235+hauntsaninja@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

KeyError: '__file__'
2 participants