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
build: fix build failures with MinGW-w64 new headers #3345
build: fix build failures with MinGW-w64 new headers #3345
Conversation
A structure definition was added to mstcpip.h in mingw-w64-x86_64-headers-git 9.0.0.6327.f29c1101f, which causes a conflict and the build fails. Fix this by changing the name in the definition in mstcpip.h.
2dedcf2
to
71d8f34
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM, but is there any benefit to putting these changes behind a MinGW #ifdef
?
What was wrong with the first attempt of directly including the necessary header? I assume that was done to support the MINGW compiler, but users are much better off going with mingw-w64 instead, or to support pre-Windows 8 compilers (when this ioctl was added), but Windows 7 support is gone from here as well. |
I don't think that has any advantages or disadvantages. This change does not do anything to MSVC. I would venture to say that putting it behind a |
Sorry for titling this without strictly distinguishing between MinGW and MinGW-w64. I think the first change will cause the build to fail when using MinGW-w64 headers before this commit (MinGW-w64 47a9f0deb), because there are no definitions such as |
Ah, understood. This is much better then, since that is quite new still. |
A structure definition was added to mstcpip.h in mingw-w64-x86_64-headers-git 9.0.0.6327.f29c1101f, which causes a conflict and the build fails. Fix this by changing the name in the definition in mstcpip.h. PR-URL: libuv#3345
A structure definition was added to mstcpip.h in mingw-w64-x86_64-headers-git 9.0.0.6327.f29c1101f, which causes a conflict and the build fails. Fix this by changing the name in the definition in mstcpip.h. PR-URL: libuv#3345
Update the libuv version to work around a mingw64 bug, refer to: libuv/libuv#3345
Update the libuv version to work around a mingw64 bug, refer to: libuv/libuv#3345
A structure definition was added to
mstcpip.h
in mingw-w64-x86_64-headers-git 9.0.0.6327.f29c1101f, which causes a conflict and the build fails. Fix this by changing the name in the definition inmstcpip.h
.Ref. msys2/MINGW-packages#9946
Build Log