Skip to content
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

Compile error in "LMS64CProtocol_remote.cpp" #355

Closed
gvanem opened this issue Dec 20, 2021 · 1 comment
Closed

Compile error in "LMS64CProtocol_remote.cpp" #355

gvanem opened this issue Dec 20, 2021 · 1 comment

Comments

@gvanem
Copy link

gvanem commented Dec 20, 2021

Compiling LMS64CProtocol_remote.cpp using MSVC, I get a long list of errors related to missing Winsock stuff:

LMS64CProtocol_remote.cpp(30): error C2065: 'WSADATA': undeclared identifier
LMS64CProtocol_remote.cpp(30): error C2146: syntax error: missing ';' before identifier 'wsaData'
LMS64CProtocol_remote.cpp(30): error C2065: 'wsaData': undeclared identifier
LMS64CProtocol_remote.cpp(31): error C2065: 'wsaData': undeclared identifier
LMS64CProtocol_remote.cpp(31): error C3861: 'WSAStartup': identifier not found
LMS64CProtocol_remote.cpp(35): error C2065: 'AF_INET': undeclared identifier
...

An easy fix for me was to patch like so:

--- a/src/ConnectionRemote/LMS64CProtocol_remote.cpp 2020-11-09 14:19:06
+++ b/src/ConnectionRemote/LMS64CProtocol_remote.cpp 2021-12-20 10:41:32
@@ -8,6 +8,7 @@
 #include <netinet/in.h>
 #include <arpa/inet.h>
 #else
+#include <winsock2.h>
 #include <windows.h>
 #endif
 #include <fcntl.h>

What's up with this? Isn't this file supposed to be for Windows?

Edit: I just discovered a -D_WIN32_LEAN_AND_MEAN had sneaked in (from %CL%). Removing that, it compiles.
       But IMHO it's better to force the use <winsock2.h> in this day and age.

@rjonaitis
Copy link
Member

Remote connection was intended for internal use only, it has been removed from compilation targets cfc53ed

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants