<div dir="ltr"><br><div class="gmail_extra"><br><br><div class="gmail_quote">On Thu, Jun 20, 2013 at 9:34 AM,  <span dir="ltr"><<a href="mailto:elson.wei@gmail.com" target="_blank">elson.wei@gmail.com</a>></span> wrote:<br>

<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"># HG changeset patch<br>
# User Wei, Elson <<a href="mailto:elson.wei@gmail.com">elson.wei@gmail.com</a>><br>
# Date 1371690961 -28800<br>
#      Thu Jun 20 09:16:01 2013 +0800<br>
# Node ID 5d7e94383feede6e5b62d4009a3877e479f72591<br>
# Parent  401b3ad26e66f6b69937e1aa808da07065c408a6<br>
gpg: treat "ERRSIG" as a valid keyid but no fingerprint<br>
<br>
diff --git a/hgext/gpg.py b/hgext/gpg.py<br>
--- a/hgext/gpg.py<br>
+++ b/hgext/gpg.py<br>
@@ -48,7 +48,6 @@<br>
                     pass<br>
         keys = []<br>
         key, fingerprint = None, None<br>
-        err = ""<br>
         for l in ret.splitlines():<br>
             # see DETAILS in the gnupg documentation<br>
             # filter the logger output<br>
@@ -56,8 +55,7 @@<br>
                 continue<br>
             l = l[9:]<br>
             if l.startswith("ERRSIG"):<br>
-                err = _("error while verifying signature")<br>
-                break<br>
+                keys.append(l.split(" ", 2)[:2] + ["", ""])<br></blockquote><div><br></div><div>Can't you move that to the other block?</div><div><br></div><div>(and please fold both patch together they are not independent)</div>

<div><br></div><div>Cheers,</div><div><br></div><div>Benoit </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
             elif l.startswith("VALIDSIG"):<br>
                 # fingerprint of the primary key<br>
                 fingerprint = l.split()[10]<br>
@@ -69,11 +67,9 @@<br>
                     keys.append(key + [fingerprint])<br>
                 key = l.split(" ", 2)<br>
                 fingerprint = None<br>
-        if err:<br>
-            return err, []<br>
         if key is not None:<br>
             keys.append(key + [fingerprint])<br>
-        return err, keys<br>
+        return keys<br>
<br>
 def newgpg(ui, **opts):<br>
     """create a new gpg instance"""<br>
@@ -119,10 +115,7 @@<br>
<br>
     data = node2txt(repo, node, version)<br>
     sig = binascii.a2b_base64(sig)<br>
-    err, keys = mygpg.verify(data, sig)<br>
-    if err:<br>
-        ui.warn("%s:%d %s\n" % (fn, ln , err))<br>
-        return None<br>
+    keys = mygpg.verify(data, sig)<br>
<br>
     validkeys = []<br>
     # warn for expired key and/or sigs<br>
_______________________________________________<br>
Mercurial-devel mailing list<br>
<a href="mailto:Mercurial-devel@selenic.com">Mercurial-devel@selenic.com</a><br>
<a href="http://selenic.com/mailman/listinfo/mercurial-devel" target="_blank">http://selenic.com/mailman/listinfo/mercurial-devel</a><br>
</blockquote></div><br></div></div>