Skip to content

1.13 test fixes #9

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

Merged
merged 16 commits into from
May 5, 2021
Merged

1.13 test fixes #9

merged 16 commits into from
May 5, 2021

Conversation

jepler
Copy link
Collaborator

@jepler jepler commented May 5, 2021

Right now, this gets through the first set of tests but has a bunch of failures on the async tests with --emit native. I'll continue to work on it.

Now all the tests pass locally. There are some interesting differences between our translate() and micropython's MP_ERROR_TEXT that create a barrier to many of the examples working; another problem has to do with the long-lived flag of the allocator.

jepler added 16 commits May 5, 2021 08:46
we changed this way back at fab634e for Rosie Pi and the merge
mangled it.  I think at this point it makes sense just to take upstream's
version again.
This caused a build error with external native code, as
MP_QSTR_protocol_stream was not available.
objtype.h got included twice, and vfs.h is unneeded.
.. this fixes the vfs_fat_ramdisk failure.
This fixes a fail in extra_coverage.py.
At least features0 works, and was useful in fixing the related native
test
This example works, and was useful when fixing a native code test.
Most of the other natmod examples do not work, however.
@jepler jepler force-pushed the 1.13_test_fixes branch from 0711956 to 716a82b Compare May 5, 2021 16:36
@tannewt tannewt merged commit 3dad2cb into tannewt:merge_1.13 May 5, 2021
@jepler jepler deleted the 1.13_test_fixes branch November 3, 2021 21:09
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.

2 participants