Skip to content

windows/uasyncio: Add support for uasyncio to windows dev variant. - #8030

Merged
dpgeorge merged 3 commits into
micropython:masterfrom
andrewleech:windows_uasyncio
Feb 7, 2022
Merged

dpgeorge merged 3 commits into
micropython:masterfrom
andrewleech:windows_uasyncio

Conversation

@andrewleech

@andrewleech andrewleech commented Nov 25, 2021 •

Copy link
Copy Markdown
Contributor

This PR enables the micropython uselect module on the windows port to allow freezing uasyncio into the dev variant.

This is built on top of #7780 so includes the commits from there too because github doesn't give any way to declare / separate dependent PR's.

This does also require #7999 to be merged first

@andrewleech
andrewleech force-pushed the windows_uasyncio branch 2 times, most recently from 4d356e8 to a4ac22a Compare November 25, 2021 03:14
@dpgeorge

Copy link
Copy Markdown
Member

@andrewleech can you please rebase this on latest master, now that all prerequisite PRs have been merged?

@dpgeorge

Copy link
Copy Markdown
Member

There's an error building the dev variant:

objmodule.obj : error LNK2001: unresolved external symbol _mp_module_uasyncio [C:\projects\micropython\ports\windows\micropython.vcxproj]
objmodule.obj : error LNK2001: unresolved external symbol _mp_module_uselect [C:\projects\micropython\ports\windows\micropython.vcxproj]

@dpgeorge

Copy link
Copy Markdown
Member

It just needs the relevant extmod files added to msvc/sources.props

Comment thread ports/windows/Makefile Outdated
@andrewleech

Copy link
Copy Markdown
Contributor Author

@dpgeorge I've fixed the last couple of issues here.

@dpgeorge
dpgeorge merged commit 6f7d6c5 into micropython:master Feb 7, 2022
@dpgeorge

dpgeorge commented Feb 7, 2022

Copy link
Copy Markdown
Member

Thank you!

@andrewleech
andrewleech deleted the windows_uasyncio branch February 7, 2022 05:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants