socklen_t already defined on mingw32-w32api-dev 5.3

[DarkWiiPlayer]

This is pretty much the same issue as luasocket has.

MinGWs ws2tcpip.h now defines socklen_t since the last version, causing an error when Luarocks tries to compile the .c files when installing luasec.

[DarkWiiPlayer]

Update: a possible fix for this is currently (not) being discussed on the luasocket repository here: lunarmodules/luasocket#303

[ewestbrook]

I don't know why you'd say "(not)". LuaSocket would like to do the right thing for users of new mingw32 versions without doing the wrong thing for users of earlier mingw32 versions. I'd imagine that LuaSec wants the same. If anyone has a good #ifdef for that, please share.

[DarkWiiPlayer]

Well, there's not really a discussion going on, which is precisely why linked it here. The way I understand it, nobody really knows an ideal way to deal with this and just deleting the typedef altogether as in my pull request may end up breaking the library on some other system. I'm not complaining that luasocket isn't instantly fixing this, I'm just annoyed that none of the users care enough to add to the discussion.

I imagine the best way to get this fixed quickly is having more feedback from users on "weird" systems with old or patched versions of MinGW, outdated windows versions, etc.

[limjiregister]

How to fix this issues quickly?

[brunoos]

I need to setup an environment to reproduce this issue.
I need to find a Windows machine and if you can help me with mingw...