]> Cypherpunks repositories - gostls13.git/commitdiff
codereview: clpatch tweaks
authorRuss Cox <rsc@golang.org>
Fri, 13 Nov 2009 02:37:39 +0000 (18:37 -0800)
committerRuss Cox <rsc@golang.org>
Fri, 13 Nov 2009 02:37:39 +0000 (18:37 -0800)
R=agl1
https://golang.org/cl/152114

lib/codereview/codereview.py

index 5f2e17d8d4c62f3ef2b427fbdbf551503179afb7..499b0061fbd09e151fbbf3490ef846a39a4304bb 100644 (file)
@@ -1352,10 +1352,14 @@ def DownloadCL(ui, repo, clname):
        try:
                data = MySend("/user_popup/" + nick, force_auth=False)
        except:
-               return None, None, "error looking up %s: %s" % (nick, ExceptionDetail())
+               ui.warn("error looking up %s: %s\n" % (nick, ExceptionDetail()))
+               cl.original_author = nick+"@needtofix"
+               return cl, diffdata, ""
        match = re.match(r"<b>(.*) \((.*)\)</b>", data)
-       if not match or match.group(2) != nick:
-               return None, None, "error looking up %s: cannot parse result" % (nick,)
+       if not match:
+               return None, None, "error looking up %s: cannot parse result %s" % (nick, repr(data))
+       if match.group(1) != nick and match.group(2) != nick:
+               return None, None, "error looking up %s: got info for %s, %s" % (nick, match.group(1), match.group(2))
        email = match.group(1)
 
        # Temporary hack until we move to the public code review server.