diff hgext/patchbomb.py @ 21727:5f98ad8fb8d9

patchbomb: reorder header insertions to clarify code

This is done as its own change to make the changes to test-patchbomb.t
easier to review.

--- a/hgext/patchbomb.py	Sat Jun 07 11:06:15 2014 -0400
+++ b/hgext/patchbomb.py	Sat Jun 07 11:32:37 2014 -0400
@@ -515,6 +515,7 @@
             m['Message-Id'] = genmsgid(m['X-Mercurial-Node'])
             if not firstpatch:
                 firstpatch = m['Message-Id']
+            m['X-Mercurial-Series-Id'] = firstpatch
         except TypeError:
             m['Message-Id'] = genmsgid('patchbomb')
         if parent:
@@ -522,10 +523,6 @@
             m['References'] = parent
         if not parent or 'X-Mercurial-Node' not in m:
             parent = m['Message-Id']
-        # For 0 of N messages, we won't have seen a patch yet, so
-        # don't try and produce an X-Mercurial-Series-Id.
-        if firstpatch:
-            m['X-Mercurial-Series-Id'] = firstpatch
 
         m['User-Agent'] = 'Mercurial-patchbomb/%s' % util.version()
         m['Date'] = email.Utils.formatdate(start_time[0], localtime=True)