diff options
| author | Zac Medico <zmedico@gentoo.org> | 2011-02-24 22:29:31 -0800 |
|---|---|---|
| committer | Zac Medico <zmedico@gentoo.org> | 2011-03-01 12:51:16 -0800 |
| commit | fe71c29eb3c0bd67886482c3cd394945e0b373e9 (patch) | |
| tree | 99b0c5810246f9288ecc277076ce2e01c2ad05e6 /pym/_emerge | |
| parent | 52c71cc0e82e3849e85594378bc321c36713521c (diff) | |
| download | portage-fe71c29eb3c0bd67886482c3cd394945e0b373e9.tar.gz portage-fe71c29eb3c0bd67886482c3cd394945e0b373e9.tar.bz2 portage-fe71c29eb3c0bd67886482c3cd394945e0b373e9.zip | |
action_sync: handle IPv6 [ip] in SYNC some more
This will fix bug #356375 some more.
Diffstat (limited to 'pym/_emerge')
| -rw-r--r-- | pym/_emerge/actions.py | 18 |
1 files changed, 14 insertions, 4 deletions
diff --git a/pym/_emerge/actions.py b/pym/_emerge/actions.py index 471adb310..ea8fc42fc 100644 --- a/pym/_emerge/actions.py +++ b/pym/_emerge/actions.py @@ -2067,13 +2067,23 @@ def action_sync(settings, trees, mtimedb, myopts, myaction): maxretries = -1 #default number of retries retries=0 - proto, user_name, hostname, port = re.split( - r"(rsync|ssh)://([^:/]+@)?(\[[:\da-fA-F]*\]|[^:/]*)(:[0-9]+)?", - syncuri, maxsplit=4)[1:5] + try: + proto, user_name, hostname, port = re.split( + r"(rsync|ssh)://([^:/]+@)?(\[[:\da-fA-F]*\]|[^:/]*)(:[0-9]+)?", + syncuri, maxsplit=4)[1:5] + except ValueError: + writemsg_level("!!! SYNC is invalid: %s\n" % syncuri, + noiselevel=-1, level=logging.ERROR) + return 1 if port is None: port="" if user_name is None: user_name="" + if re.match(r"^\[[:\da-fA-F]*\]$", hostname) is None: + getaddrinfo_host = hostname + else: + # getaddrinfo needs the brackets stripped + getaddrinfo_host = hostname[1:-1] updatecache_flg=True all_rsync_opts = set(rsync_opts) extra_rsync_opts = portage.util.shlex_split( @@ -2092,7 +2102,7 @@ def action_sync(settings, trees, mtimedb, myopts, myaction): try: addrinfos = getaddrinfo_validate( - socket.getaddrinfo(hostname, None, + socket.getaddrinfo(getaddrinfo_host, None, family, socket.SOCK_STREAM)) except socket.error as e: writemsg_level( |