Not that I know much about Cygwin or mingw…

Jon TURNEY <[email protected]> (27/04/2011):
> For the Cygwin target, we will be using the socket functions provided
> by the Cygwin DLL, so linking with ws2_32 is unnecessary, even though
> it may be present.
> 
> Signed-off-by: Jon TURNEY <[email protected]>
> ---
>  configure.ac |    9 ++++++++-
>  1 files changed, 8 insertions(+), 1 deletions(-)
> 
> diff --git a/configure.ac b/configure.ac
> index d18b0fe..84a9f4f 100644
> --- a/configure.ac
> +++ b/configure.ac
> @@ -86,7 +86,14 @@ case $host_os in
>       ;;
>  esac
>  
> -AC_HAVE_LIBRARY(ws2_32)
> +# Link with winsock if mingw target
> +case $host_os in
> +     *mingw*)
> +             AC_HAVE_LIBRARY(ws2_32)
> +     ;;
> +     *)
> +     ;;
> +esac

it does look pretty harmless for the archs I care for, so you can add my:

Reviewed-by: Cyril Brulebois <[email protected]>

Mraw,
KiBi.

Attachment: signature.asc
Description: Digital signature

_______________________________________________
[email protected]: X.Org development
Archives: http://lists.x.org/archives/xorg-devel
Info: http://lists.x.org/mailman/listinfo/xorg-devel

Reply via email to