-
-
Notifications
You must be signed in to change notification settings - Fork 8.2k
Windows port does not build in Visual Studio 2015 #1532
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
Comments
It never never did, not it's supposed to. @stinos : Please remove any star-includes from MSVC project files. |
… containing optional features Fixes issues like micropython#1532
see #1535 |
… containing optional features Fixes issues like #1532
@stinos : Thanks. Can you also do something about this: #1513 (comment) , as this same issue appears to be tackled in the patch above too. |
Excellent, now I'm able to build without using any of the /lib components. I cleaned up the necessary changes for the build in Visual Studio 2015 to work and opened a pull request: #1538 |
Assuming fixed. |
I am trying to get a Micropython binary for Windows, and there are many issues with the existing port. I have made some changes that allow me to build the project, but linking fails miserably. Unfortunately, this is way out of my comfort zone so I am not able to figure out what is failing at the linking step.
My changes are summarized here: https://github.com/omtinez/micropython/commit/9ac7a21b6dc70b436064b9d5d5486b3cd08aaf21.
Some things worth noting:
lwip-contrib
repo into/micropython/windows/lwip-win32
ip_addr
in lwIP are completely broken, since they all point toipv4/lwip
instead of directlylwip
. I had to do a mass string-replace to get the building to work, and that is not reflected as part of the commit linked above since it's in a different repository (where does exactly the lib folder come from to get populated by thegit submodule update --init
command?The output of the linker is the following:
The text was updated successfully, but these errors were encountered: