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

Test with 32-bit Python #14634

Merged
merged 1 commit into from
Feb 15, 2023
Merged

Test with 32-bit Python #14634

merged 1 commit into from
Feb 15, 2023

Conversation

mr-c
Copy link
Contributor

@mr-c mr-c commented Feb 7, 2023

Debian does build 32bit (on Linux), and there was a recent regression

This PR would have caught #14633 earlier

No change in total CI time (32 minutes) versus the baseline (33 minutes)

Confirmation that the new CI test catches the previous error is at https://github.com/python/mypy/actions/runs/4174055572/jobs/7227150570#step:7:44

@github-actions

This comment has been minimized.

@JukkaL
Copy link
Collaborator

JukkaL commented Feb 7, 2023

Yeah, we should probably continue testing on a 32-bit architecture. Maybe just running mypyc tests would be enough to catch most issues? Windows CI tends to be pretty slow.

@mr-c
Copy link
Contributor Author

mr-c commented Feb 7, 2023

Yeah, we should probably continue testing on a 32-bit architecture. Maybe just running mypyc tests would be enough to catch most issues? Windows CI tends to be pretty slow.

That or cross-compiling to x86 under Linux; whichever works!

@JukkaL
Copy link
Collaborator

JukkaL commented Feb 7, 2023

That or cross-compiling to x86 under Linux; whichever works!

I'm not sure how easy this would be to set up in GitHub actions. We might need to use Docker, which may be too involved. Continuing to use 32-bit Windows may be our best bet, but without running the entire test suite.

@github-actions

This comment has been minimized.

@mr-c
Copy link
Contributor Author

mr-c commented Feb 14, 2023

I rebased on top of @JukkaL 's 32-bit test case fixes in 563e29d

There are some failures, but I think they are just from path changes
https://github.com/python/mypy/actions/runs/4172280556/jobs/7223187539#step:8:202

FAILED mypy/test/testfinegrained.py::FineGrainedSuite::fine-grained-suggest.test::testSuggestColonMethodJSON

Alignment of first line difference:
  E: ..., "line": 3, "path": "tmp/foo.py", "samples": 0, "signature": {"arg_t...
  A: ..., "line": 3, "path": "C:/Users/RUNNER~1/AppData/Local/Temp/mypy-test-...
                              ^
FAILED mypyc/test/test_run.py::TestRun::run-loops.test::testForIterable

Alignment of first line difference:
  E:   File "driver.py", line 16, in <module>...
  A:   File "C:\Users\RUNNER~1\AppData\Local\Temp\mypy-test-2afcrxij\tmp\driv...
             ^
FAILED mypyc/test/test_run.py::TestRun::run-exceptions.test::testException

Alignment of first line difference:
  E:   File "driver.py", line 4, in <module>...
  A:   File "C:\Users\RUNNER~1\AppData\Local\Temp\mypy-test-63wapg7j\tmp\driv...
             ^
FAILED mypyc/test/test_run.py::TestRun::run-exceptions.test::testTryExcept

Alignment of first line difference:
  E:   File "driver.py", line 6, in <module>...
  A:   File "C:\Users\RUNNER~1\AppData\Local\Temp\mypy-test-kx2rlobh\tmp\driv...
             ^
FAILED mypyc/test/test_run.py::TestRun::run-exceptions.test::testExceptionAtModuleTopLevel

Alignment of first line difference:
  E:   File "driver.py", line 3, in <module>...
  A:   File "C:\Users\RUNNER~1\AppData\Local\Temp\mypy-test-njz3rjwc\tmp\driv...
             ^
FAILED mypyc/test/test_run.py::TestRun::run-classes.test::testSubclassUninitAttr

 Alignment of first line difference:
  E:   File "driver.py", line 4, in <module>...
  A:   File "C:\Users\RUNNER~1\AppData\Local\Temp\mypy-test-hb3g0dzm\tmp\driv...
             ^
FAILED mypyc/test/test_run.py::TestRun::run-generators.test::testYieldThrow

 Alignment of first line difference:
  E:   File "driver.py", line 31, in <module>...
  A:   File "C:\Users\RUNNER~1\AppData\Local\Temp\mypy-test-voe0j96t\tmp\driv...
             ^
FAILED mypyc/test/test_run.py::TestRun::run-classes.test::testProperty


Alignment of first line difference:
  E:   File "driver.py", line 5, in <module>...
  A:   File "C:\Users\RUNNER~1\AppData\Local\Temp\mypy-test-45xmabsc\tmp\driv...
             ^
FAILED mypyc/test/test_run.py::TestRun::run-multimodule.test::testMultiModuleTraceback

 Alignment of first line difference:
  E:   File "driver.py", line 6, in <module>...
  A:   File "C:\Users\RUNNER~1\AppData\Local\Temp\mypy-test-cacveqm3\tmp\driv...
             ^
FAILED mypyc/test/test_run.py::TestRunMultiFile::run-multimodule.test::testMultiModuleTraceback_multi


Alignment of first line difference:
  E:   File "driver.py", line 6, in <module>...
  A:   File "C:\Users\RUNNER~1\AppData\Local\Temp\mypy-test-_g8qvv4c\tmp\driv...
             ^
FAILED mypyc/test/test_run.py::TestRunSeparate::run-multimodule.test::testMultiModuleTraceback_separate


Alignment of first line difference:
  E:   File "driver.py", line 6, in <module>...
  A:   File "C:\Users\RUNNER~1\AppData\Local\Temp\mypy-test-k8mxhwba\tmp\driv...
             ^

@mr-c mr-c force-pushed the test-x86 branch 5 times, most recently from 79ddeeb to 8554dbb Compare February 14, 2023 11:51
@mr-c mr-c changed the title Test with 32-bit on Windows again Test with 32-bit Python Feb 14, 2023
@github-actions

This comment has been minimized.

@mr-c mr-c force-pushed the test-x86 branch 3 times, most recently from 75a4157 to a1aaa87 Compare February 14, 2023 13:03

Verified

This commit was created on GitHub.com and signed with GitHub’s verified signature. The key has expired.
Debian does build 32bit and there was a recent regression
@mr-c mr-c marked this pull request as ready for review February 14, 2023 13:19
@mr-c
Copy link
Contributor Author

mr-c commented Feb 14, 2023

@JukkaL I added 32-bit Linux testing (instead of MS Windows), updated the description, and marked this PR as ready for review

@github-actions
Copy link
Contributor

According to mypy_primer, this change has no effect on the checked open source code. 🤖🎉

Copy link
Collaborator

@hauntsaninja hauntsaninja left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks!

@hauntsaninja hauntsaninja merged commit 0bbeab8 into python:master Feb 15, 2023
@mr-c mr-c deleted the test-x86 branch February 15, 2023 22:25
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.

None yet

3 participants