Re: [Mingw-w64-public] [PATCH] netioapi.h: Include ws2ipdef.h.
Hi Jacek, On Dec 15 12:06, Jacek Caban wrote: > Hi Corinna, > > On 12/15/14 11:59, Corinna Vinschen wrote: > > Hi guys, > > > > On Oct 15 20:57, Kai Tietz wrote: > >> Patch is ok. Please go ahead and apply. > >> > >> Thanks, > >> Kai > >> > >> 2014-10-15 20:14 GMT+02:00 Jacek Caban : > >>> For SOCKADDR_INET. > >>> > >>> --- > >>> mingw-w64-headers/include/netioapi.h | 1 + > >>> 1 file changed, 1 insertion(+) > > This patch breaks Cygwin applications using the IP Helper API. > > iphlpapi.h includes netioapi.h which now includes ws2ipdef.h. See > > https://cygwin.com/ml/cygwin/2014-12/msg00198.html for a description. > > Typically we don't want to include ws2ipdef.h and winsock2.h from Cygwin > > applications becasue Winsock sockets and their definitions collide with > > Cygwin POSIX sockets and their definitions. The IP helper API needs to > > be separate. > > > > Is it ok to simply revert the patch for Cygwin: > > The patch is fine with me. However, to not have similar problems in the > future, maybe making problematic headers no-op on Cygwin would be a > better solution than avoiding including them? I'm not sure. For historical reasons, Cygwin applications can still opt for using Winsock sockets instead of Cygwin sockets. I'm not sure how much sense is in there, but the combination of various macros (like the dreaded __USE_W32_SOCKETS) is a bit fragile and I wouldn't want to break that. Also, typically this only affects Winsock headers. With other Windows headers we typically don't have collision problems(*). Thanks, Corinna (*) There's a type collision with the GNU regex implementation but it's pretty simple to workaround and the code using it usually doesn't include Windows headers. pgpkFE8dfaidr.pgp Description: PGP signature -- Download BIRT iHub F-Type - The Free Enterprise-Grade BIRT Server from Actuate! Instantly Supercharge Your Business Reports and Dashboards with Interactivity, Sharing, Native Excel Exports, App Integration & more Get technology previously reserved for billion-dollar corporations, FREE http://pubads.g.doubleclick.net/gampad/clk?id=164703151&iu=/4140/ostg.clktrk___ Mingw-w64-public mailing list Mingw-w64-public@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/mingw-w64-public
Re: [Mingw-w64-public] [PATCH] netioapi.h: Include ws2ipdef.h.
Hi Corinna, On 12/15/14 11:59, Corinna Vinschen wrote: > Hi guys, > > On Oct 15 20:57, Kai Tietz wrote: >> Patch is ok. Please go ahead and apply. >> >> Thanks, >> Kai >> >> 2014-10-15 20:14 GMT+02:00 Jacek Caban : >>> For SOCKADDR_INET. >>> >>> --- >>> mingw-w64-headers/include/netioapi.h | 1 + >>> 1 file changed, 1 insertion(+) > This patch breaks Cygwin applications using the IP Helper API. > iphlpapi.h includes netioapi.h which now includes ws2ipdef.h. See > https://cygwin.com/ml/cygwin/2014-12/msg00198.html for a description. > Typically we don't want to include ws2ipdef.h and winsock2.h from Cygwin > applications becasue Winsock sockets and their definitions collide with > Cygwin POSIX sockets and their definitions. The IP helper API needs to > be separate. > > Is it ok to simply revert the patch for Cygwin: The patch is fine with me. However, to not have similar problems in the future, maybe making problematic headers no-op on Cygwin would be a better solution than avoiding including them? Jacek -- Download BIRT iHub F-Type - The Free Enterprise-Grade BIRT Server from Actuate! Instantly Supercharge Your Business Reports and Dashboards with Interactivity, Sharing, Native Excel Exports, App Integration & more Get technology previously reserved for billion-dollar corporations, FREE http://pubads.g.doubleclick.net/gampad/clk?id=164703151&iu=/4140/ostg.clktrk ___ Mingw-w64-public mailing list Mingw-w64-public@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/mingw-w64-public
Re: [Mingw-w64-public] [PATCH] netioapi.h: Include ws2ipdef.h.
Hi guys, On Oct 15 20:57, Kai Tietz wrote: > Patch is ok. Please go ahead and apply. > > Thanks, > Kai > > 2014-10-15 20:14 GMT+02:00 Jacek Caban : > > For SOCKADDR_INET. > > > > --- > > mingw-w64-headers/include/netioapi.h | 1 + > > 1 file changed, 1 insertion(+) This patch breaks Cygwin applications using the IP Helper API. iphlpapi.h includes netioapi.h which now includes ws2ipdef.h. See https://cygwin.com/ml/cygwin/2014-12/msg00198.html for a description. Typically we don't want to include ws2ipdef.h and winsock2.h from Cygwin applications becasue Winsock sockets and their definitions collide with Cygwin POSIX sockets and their definitions. The IP helper API needs to be separate. Is it ok to simply revert the patch for Cygwin: diff --git a/mingw-w64-headers/include/netioapi.h b/mingw-w64-headers/include/netioapi.h index 9e7f67d..164f972 100644 --- a/mingw-w64-headers/include/netioapi.h +++ b/mingw-w64-headers/include/netioapi.h @@ -7,7 +7,9 @@ #define _INC_NETIOAPI #include +#ifndef __CYGWIN__ #include +#endif #include #include #include Thanks, Corinna pgpsiDHGYUdB4.pgp Description: PGP signature -- Download BIRT iHub F-Type - The Free Enterprise-Grade BIRT Server from Actuate! Instantly Supercharge Your Business Reports and Dashboards with Interactivity, Sharing, Native Excel Exports, App Integration & more Get technology previously reserved for billion-dollar corporations, FREE http://pubads.g.doubleclick.net/gampad/clk?id=164703151&iu=/4140/ostg.clktrk___ Mingw-w64-public mailing list Mingw-w64-public@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/mingw-w64-public
Re: [Mingw-w64-public] [PATCH] netioapi.h: Include ws2ipdef.h.
Patch is ok. Please go ahead and apply. Thanks, Kai 2014-10-15 20:14 GMT+02:00 Jacek Caban : > For SOCKADDR_INET. > > --- > mingw-w64-headers/include/netioapi.h | 1 + > 1 file changed, 1 insertion(+) > > > > -- > Comprehensive Server Monitoring with Site24x7. > Monitor 10 servers for $9/Month. > Get alerted through email, SMS, voice calls or mobile push notifications. > Take corrective actions from your mobile device. > http://p.sf.net/sfu/Zoho > ___ > Mingw-w64-public mailing list > Mingw-w64-public@lists.sourceforge.net > https://lists.sourceforge.net/lists/listinfo/mingw-w64-public > -- Comprehensive Server Monitoring with Site24x7. Monitor 10 servers for $9/Month. Get alerted through email, SMS, voice calls or mobile push notifications. Take corrective actions from your mobile device. http://p.sf.net/sfu/Zoho ___ Mingw-w64-public mailing list Mingw-w64-public@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/mingw-w64-public