dlls/ws2_32/socket.c missing a #ifdef in 1.7.40

[ischou]

At around line 366:

Code: Select all

        DEBUG_SOCKLEVEL(WS_NSPROTO_IPX);
        switch(optname)
        {
            DEBUG_SOCKOPT(WS_IPX_PTYPE);
            DEBUG_SOCKOPT(WS_IPX_FILTERPTYPE);
            DEBUG_SOCKOPT(WS_IPX_DSTYPE);
            DEBUG_SOCKOPT(WS_IPX_RECVHDR);
            DEBUG_SOCKOPT(WS_IPX_MAXSIZE);
            DEBUG_SOCKOPT(WS_IPX_ADDRESS);
            DEBUG_SOCKOPT(WS_IPX_MAX_ADAPTER_NUM);
        }
        break;

All the WS_IPX_* are not defined if HAS_IPX was not defined, so should this whole block get surrounded with an ifdef HAS_IPX/endif?

Causes a clean build of 1.7.40 to fail for me.

[olivierfrdierick]

Wine devs are seldom seen on these forums.
You should file a bug report "Wine compilation fails when HAS_IPX is not defined", specifying how you did get HAS_IPX undefined and the exact make errors.
I'm sure a dev will make a patch quickly if it's as easy as adding an #ifdef.

[doh123]

I just noticed I have this same issue…

EDIT:
I file a bug report and attached a patchfile that fixed the issue for me.
https://bugs.winehq.org/show_bug.cgi?id=38410