[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH] gethostname: fix build on mingw
From: |
Matthias Bolte |
Subject: |
Re: [PATCH] gethostname: fix build on mingw |
Date: |
Sat, 20 Mar 2010 00:35:13 +0100 |
2010/3/20 Eric Blake <address@hidden>:
> Since commit 4e86671a, gethostname failed to compile on mingw.
> Gnulib's sys/socket.h includes winsock2.h, which then includes
> unistd.h prior to declaring gethostname. Therefore, unistd.h
> cannot replace gethostname until winsock2.h is complete.
>
> * lib/sys_socket.in.h (includes): Set witness when including
> winsock2.h.
> * lib/unistd.in.h (includes): Don't replace anything when included
> by winsock2.h.
> Reported by Matthias Bolte.
>
> Signed-off-by: Eric Blake <address@hidden>
> ---
>
> Thanks to Matthias for doing the git bisect to narrow down
> the cause of his error:
>
> # i686-pc-mingw32-gcc -DHAVE_CONFIG_H -I. -I.. -g -O2 -MT sockets.o
> -MD -MP -MF -deps/sockets.Tpo -c -o sockets.o sockets.c
> In file included from ./sys/socket.h:389,
> from sockets.h:37,
> from sockets.c:23:
> /usr/i686-pc-mingw32/sys-root/mingw/include/winsock2.h:635: error:
> conflicting types for 'rpl_gethostname'
> ./unistd.h:824: note: previous declaration of 'rpl_gethostname' was here
>
> I believe this will solve the issue, but would appreciate some testing
> by Matthias. I also think that this is the same issue as:
> http://lists.gnu.org/archive/html/bug-gnulib/2010-01/msg00183.html
>
I tested this patch and can confirm that this patch fixes the
gethostname compile issue on MinGW. Thanks.
Matthias
- [PATCH] gethostname: fix build on mingw, Eric Blake, 2010/03/19
- Re: [PATCH] gethostname: fix build on mingw,
Matthias Bolte <=
- Re: [PATCH] gethostname: fix build on mingw, Eric Blake, 2010/03/20
- Re: [PATCH] gethostname: fix build on mingw, Matthias Bolte, 2010/03/21
- Re: [PATCH] gethostname: fix build on mingw, Eric Blake, 2010/03/22
- [PATCHv4] gethostname: fix build on mingw, Eric Blake, 2010/03/22
- Re: [PATCHv4] gethostname: fix build on mingw, Simon Josefsson, 2010/03/22
- Re: [PATCHv4] gethostname: fix build on mingw, Eric Blake, 2010/03/22
- Re: [PATCHv4] gethostname: fix build on mingw, Simon Josefsson, 2010/03/22