[MERGE] [#165014] Support IPv6 in smart server
Andrew Bennetts
andrew at canonical.com
Wed Sep 3 00:17:40 BST 2008
Andrew Bennetts has voted tweak.
Status is now: Conditionally approved
Comment:
This looks ok to me, but I wonder if it works on all platforms?
Specifically, I'm not sure that using None rather than '0.0.0.0' as the
host for sock.bind((host, port)) is cross-platform? The Python 2.4 docs
say host can be an IP address or domain name, but doesn't mention None.
Also, I'm not sure if we can always expect constants like AF_UNSPEC and
AI_ADDRCONFIG to be available on all platforms?
Basically, if this works on Windows, I'm happy. So the "tweak" I'm
asking for is confirmation that someone has tried this patch on Windows.
For details, see:
http://bundlebuggy.aaronbentley.com/project/bzr/request/%3C20080830185330.GA22976%40vernstok.nl%3E
Project: Bazaar
More information about the bazaar
mailing list