comparison tests/run-tests.py @ 30985:1f803482844a

runtests: checkportisavailable should only check one family

As explained by the previous patch, checkportisavailable() should only check
the preferred family - either IPv4 or IPv6, not both.

This patch makes it so.

30984:15f9084a9a0c

useipv6 = checkipv6available()

def checkportisavailable(port):
    """return true if a port seems free to bind on localhost"""
    families = [getattr(socket, i, None)
                for i in ('AF_INET', 'AF_INET6')
                if getattr(socket, i, None) is not None]
    for family in families:
        try:
            s = socket.socket(family, socket.SOCK_STREAM)
            s.bind(('localhost', port))
            s.close()
            return True

30985:1f803482844a

useipv6 = checkipv6available()

def checkportisavailable(port):
    """return true if a port seems free to bind on localhost"""
    if useipv6:
        family = socket.AF_INET6
    else:
        family = socket.AF_INET
    if True:
        try:
            s = socket.socket(family, socket.SOCK_STREAM)
            s.bind(('localhost', port))
            s.close()
            return True