Re: [PATCH] Add IPv6 support for remote TCP connections

Eli Zaretskii
2014-02-09T16:35:01Z

> Date: Sun, 9 Feb 2014 14:05:01 +0100
> From: Jan Kratochvil <jan.kratochvil@redhat.com>
> Cc: gdb-patches@sourceware.org, Kai Tietz <ktietz@redhat.com>
>
> On Sun, 09 Feb 2014 10:53:08 +0100, Paul Fertser wrote:
> > Do you take into account that there're currently two competing
> > solutions for providing windows support: MinGW and MinGW-w64?
>
> Yes, recent Fedoras AFAIK follow the MinGW-w64 port:
> https://fedoraproject.org/wiki/MinGW?rd=SIGs/MinGW
> https://fedoraproject.org/wiki/MinGW/CrossCompilerFramework
> http://mingw-w64.sourceforge.net/
>
>
> > I've just tried cross-compiling code with getaddrinfo with mingw-w64
> > without any replacements and it was built (and run with wine) just
> > fine.
>
> I have different results for gdb-7.7 with your patch on Fedora Rawhide
> (=F-21pre) x86_64:
>
> ../../gdb/ser-tcp.c: In function 'net_open':
> ../../gdb/ser-tcp.c:162:19: error: storage size of 'hints' isn't known
> struct addrinfo hints;
> ^
> ../../gdb/ser-tcp.c:196:30: error: invalid application of 'sizeof' to incomplete type 'struct addrinfo'
> memset (&hints, 0, sizeof (struct addrinfo));
> ^
> ../../gdb/ser-tcp.c:205:3: warning: implicit declaration of function 'getaddrinfo' [-Wimplicit-function-declaration]
> tmp = getaddrinfo (hostname, port_str, &hints, &result);
> ^
> [...]

Did you include ws2tcpip.h? That's the header where getaddrinfo and
all that is needed to use it are declared, per the MSDN documentation: