Skip to content

Conversation

@ccawley2011
Copy link
Contributor

No description provided.

@sezero
Copy link
Contributor

sezero commented Jan 30, 2022

Why do we truly need this?

@sezero
Copy link
Contributor

sezero commented Jan 30, 2022

I mean, the windows version relies on iphlpapi.dll even in the SDL-1.2
branch which isn't available (at least by default as far as I know) on
w9x/nt4, so we certainly shouldn't care about winsock-1.x.

Closing. @slouken, @icculus: reopen and/or merge this if you disagree.

@sezero sezero closed this Jan 30, 2022
@ccawley2011
Copy link
Contributor Author

Why do we truly need this?

This is needed to support Windows 95 without the Winsock 2 upgrade installed.

I mean, the windows version relies on iphlpapi.dll even in the SDL-1.2 branch which isn't available (at least by default as far as I know) on w9x/nt4, so we certainly shouldn't care about winsock-1.x.

iphlpapi.dll is included with newer versions of Internet Explorer (which doesn't guarantee the existence of the Winsock 2 upgrade). In a Winsock 2 build, it'll be necessary to replace GetAdaptersInfo for UWP builds anyway (see issue #30), and while it would still be necessary to have GetAdaptersInfo in Winsock 1 builds, it would be better to address it in a separate PR.

@sezero
Copy link
Contributor

sezero commented Jan 30, 2022

OK then, leaving decision to @slouken and/or @icculus

@sezero sezero reopened this Jan 30, 2022
@sezero
Copy link
Contributor

sezero commented Jan 30, 2022

If this gets accepted though, note that one issue is this: #26
(original report at mxe/mxe#550), i.e.: iphlpapi.h needing ws2tcpip.h with some SDKs.

@slouken
Copy link
Collaborator

slouken commented Jun 16, 2022

We don't officially support Windows 95, but I think this is harmless. @ccawley2011, can you fix this PR so you don't reintroduce #26?

@icculus
Copy link
Collaborator

icculus commented Aug 7, 2025

If someone wants to decide this is worth merging or updating to be merged, please speak up before I unceremoniously close this PR. I'm fine with it going in, but if it breaks mingw or whatever, we're never going to know.

@icculus
Copy link
Collaborator

icculus commented Aug 9, 2025

Closing this. If someone wants to take another run at it, I'm fine with it, but it's probably fine to let SDL_net 1.2 (and Windows 95) alone.

@icculus icculus closed this Aug 9, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

4 participants