patchbomb: do not assume the presence of angle brackets around msg-id
authorCédric Duval <cedricduval@free.fr>
Wed, 17 Jun 2009 20:54:26 +0200
changeset 8826 2aff285b902f
parent 8825 ec33cf8610a9
child 8827 34d611dbed56
patchbomb: do not assume the presence of angle brackets around msg-id RFC 5322 states: "Semantically, the angle bracket characters are not part of the msg-id; the msg-id is what is contained between the two angle bracket characters." Hence it should be correct to pass a message Id with no angle brackets to --in-reply-to. Adding them if missing.
hgext/patchbomb.py
--- a/hgext/patchbomb.py	Thu Jun 18 01:21:26 2009 +0100
+++ b/hgext/patchbomb.py	Wed Jun 17 20:54:26 2009 +0200
@@ -397,6 +397,13 @@
     ui.write('\n')
 
     parent = opts.get('in_reply_to') or None
+    # angle brackets may be omitted, they're not semantically part of the msg-id
+    if parent is not None:
+        if not parent.startswith('<'):
+            parent = '<' + parent
+        if not parent.endswith('>'):
+            parent += '>'
+
     first = True
 
     sender_addr = email.Utils.parseaddr(sender)[1]