You are right, VC6 does support IP Helper API with recent SDK. OS support: Michael pointed out the OS requirements for the API (http://sourceforge.net/mailarchive/message.php?msg_id=38034625) - there is no need for WINVER checks.
Should the VC6 project files be supported in parallel to VC2005 files or eventually dropped? If supported in parallel, we may add a line to win32/README.txt about potential problems with iphlpapi.h on ancient SDKs and patch VC6 files as well: { hunk ./win32/README.txt 11 + +If you build with Visual Studio 6, please make sure you use recent SDK +(vanilla Visual Studio 6 is missing iphlpapi.h). hunk ./win32/config.h.in 155 -#undef HAVE_IPHLPAPI_H +#define HAVE_IPHLPAPI_H 1 hunk ./win32/libsofia-sip-ua/libsofia_sip_ua.dsp 56 -# ADD LINK32 kernel32.lib user32.lib ws2_32.lib advapi32.lib /nologo /dll /machine:I386 +# ADD LINK32 kernel32.lib user32.lib ws2_32.lib iphlpapi.lib advapi32.lib /nologo /dll /machine:I386 hunk ./win32/libsofia-sip-ua/libsofia_sip_ua.dsp 83 -# ADD LINK32 kernel32.lib user32.lib ws2_32.lib advapi32.lib /nologo /dll /debug /machine:I386 /pdbtype:sept +# ADD LINK32 kernel32.lib user32.lib ws2_32.lib iphlpapi.lib advapi32.lib /nologo /dll /debug /machine:I386 /pdbtype:sept hunk ./win32/libsofia-sip-ua/libsofia_sip_ua.vcproj 79 - AdditionalDependencies="ws2_32.lib advapi32.lib" + AdditionalDependencies="ws2_32.lib advapi32.lib iphlpapi.lib" hunk ./win32/libsofia-sip-ua/libsofia_sip_ua.vcproj 179 - AdditionalDependencies="ws2_32.lib" + AdditionalDependencies="ws2_32.lib iphlpapi.lib" hunk ./win32/libsofia-sip-ua-static/libsofia_sip_ua_static.vcproj 73 + AdditionalDependencies="iphlpapi.lib" hunk ./win32/libsofia-sip-ua-static/libsofia_sip_ua_static.vcproj 151 + AdditionalDependencies="iphlpapi.lib" } If no, Kai should get the recent SDK somehow :) (http://sourceforge.net/mailarchive/forum.php?thread_id=31477613&forum_id=45790) Roman Pekka Pessi wrote: > On 2/6/07, Michael Jerris <[EMAIL PROTECTED]> wrote: >> > hunk ./win32/config.h.in 155 >> > -#undef HAVE_IPHLPAPI_H >> > +#define HAVE_IPHLPAPI_H 1 >> >> If you wrap this in #if _MSC_VER > 1200 this should work depending on >> the version. There may be a better way to check this, but I know it >> will be present on the later versions, the rest of the patch looks good >> to me. > > If it is the platform sdk feature, I'd guess that even VC6 could > support it given modern enough sdk. But is it in 98 or on? Should we > have something like > > #define HAVE_IPHLPAPI_H (defined(WINVER) && WINVER >= 0x0500)) > > or > > #define HAVE_IPHLPAPI_H (defined(WINVER) && WINVER >= 0x0501)) > > But tt looks like the macro expansion behaves funny in vc, this is > always false??? ------------------------------------------------------------------------- Using Tomcat but need to do more? Need to support web services, security? Get stuff done quickly with pre-integrated technology to make your job easier. Download IBM WebSphere Application Server v.1.0.1 based on Apache Geronimo http://sel.as-us.falkag.net/sel?cmd=lnk&kid=120709&bid=263057&dat=121642 _______________________________________________ Sofia-sip-devel mailing list Sofia-sip-devel@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/sofia-sip-devel