nisse@lysator.liu.se (Niels Möller) writes:
Or are you afraid that the code will break on systems with no ipv6? To test for ipv6 availability in libc it might be good to check for AF_INET6, but I think the tests for getaddrinfo and friends will weed out such systems anyway, and disable ipv6 for them.
Bingo.
There is at least one implementation of getaddrinfo for IPv4 (URL:http://www.sra.co.jp/people/m-kasahr/getaddrinfo/, and while I don't think it would be a disaster if compilation breaks for those system (which I'm actually not sure it does, it seems to have dummy definition of the inet6-stuff, but I don't know whatever it installs it).
What we gain by removing the test:
* A quicker ./configure
What we may possibly lose:
* Broken IPv6 capable systems where lsh won't compile might still try to use IPv6 and cause errors.
* Breaks compilations on IPv4 systems with getaddrinfo and friends.
I wouldn't bet that there aren't any systems without AF_INET6 but with getaddrinfo, so I think it's best to keep the test.
/Pontus