cURL / Mailing Lists / curl-library / Single Mail

curl-library

Re: [PATCH] cmake: enable IPv6 by default if available

From: Daniel Stenberg <daniel_at_haxx.se>
Date: Mon, 13 Oct 2014 11:17:01 +0200 (CEST)

On Sun, 12 Oct 2014, Peter Wu wrote:

> The cache hack is needed because ENABLE_IPV6 is a macro guard. HAVE_IPV6
> would be a nicer name, but is a more evasive change.

I think I originally went with "enable" just because people can in fact
disable this so it is a matter of enabling the use of IPv6, not just a matter
if you actually have support for it. The HAVE_* defines are usually the latter
kind.

Of course, over time and these days, I think the amount of people who
explicitly disable IPv6 is very small.

I don't think replacing the definition name offers any particular benefit.

-- 
  / daniel.haxx.se
-------------------------------------------------------------------
List admin: http://cool.haxx.se/list/listinfo/curl-library
Etiquette:  http://curl.haxx.se/mail/etiquette.html
Received on 2014-10-13