Mercurial > hg-stable
annotate contrib/patchbomb @ 1135:e455d91f6259
Variable 'body' was missing in patchbomb script.
author | Thomas Arendsen Hein <thomas@intevation.de> |
---|---|
date | Sun, 28 Aug 2005 18:52:55 +0200 |
parents | 63b5f68d8167 |
children | d451888505d7 |
rev | line source |
---|---|
875 | 1 #!/usr/bin/python |
2 # | |
3 # Interactive script for sending a collection of Mercurial changesets | |
4 # as a series of patch emails. | |
5 # | |
6 # The series is started off with a "[PATCH 0 of N]" introduction, | |
7 # which describes the series as a whole. | |
8 # | |
9 # Each patch email has a Subject line of "[PATCH M of N] ...", using | |
10 # the first line of the changeset description as the subject text. | |
11 # The message contains two or three body parts: | |
12 # | |
13 # The remainder of the changeset description. | |
14 # | |
877
25430c523677
Polish patchbomb script.
Bryan O'Sullivan <bos@serpentine.com>
parents:
876
diff
changeset
|
15 # [Optional] If the diffstat program is installed, the result of |
25430c523677
Polish patchbomb script.
Bryan O'Sullivan <bos@serpentine.com>
parents:
876
diff
changeset
|
16 # running diffstat on the patch. |
875 | 17 # |
18 # The patch itself, as generated by "hg export". | |
19 # | |
20 # Each message refers to all of its predecessors using the In-Reply-To | |
21 # and References headers, so they will show up as a sequence in | |
22 # threaded mail and news readers, and in mail archives. | |
23 # | |
24 # For each changeset, you will be prompted with a diffstat summary and | |
25 # the changeset summary, so you can be sure you are sending the right | |
26 # changes. | |
27 # | |
28 # It is best to run this script with the "-n" (test only) flag before | |
877
25430c523677
Polish patchbomb script.
Bryan O'Sullivan <bos@serpentine.com>
parents:
876
diff
changeset
|
29 # firing it up "for real", in which case it will use your pager to |
25430c523677
Polish patchbomb script.
Bryan O'Sullivan <bos@serpentine.com>
parents:
876
diff
changeset
|
30 # display each of the messages that it would send. |
875 | 31 # |
32 # To configure a default mail host, add a section like this to your | |
33 # hgrc file: | |
34 # | |
35 # [smtp] | |
36 # host = my_mail_host | |
37 # port = 1025 | |
877
25430c523677
Polish patchbomb script.
Bryan O'Sullivan <bos@serpentine.com>
parents:
876
diff
changeset
|
38 # |
25430c523677
Polish patchbomb script.
Bryan O'Sullivan <bos@serpentine.com>
parents:
876
diff
changeset
|
39 # To configure other defaults, add a section like this to your hgrc |
25430c523677
Polish patchbomb script.
Bryan O'Sullivan <bos@serpentine.com>
parents:
876
diff
changeset
|
40 # file: |
25430c523677
Polish patchbomb script.
Bryan O'Sullivan <bos@serpentine.com>
parents:
876
diff
changeset
|
41 # |
25430c523677
Polish patchbomb script.
Bryan O'Sullivan <bos@serpentine.com>
parents:
876
diff
changeset
|
42 # [patchbomb] |
25430c523677
Polish patchbomb script.
Bryan O'Sullivan <bos@serpentine.com>
parents:
876
diff
changeset
|
43 # from = My Name <my@email> |
25430c523677
Polish patchbomb script.
Bryan O'Sullivan <bos@serpentine.com>
parents:
876
diff
changeset
|
44 # to = recipient1, recipient2, ... |
25430c523677
Polish patchbomb script.
Bryan O'Sullivan <bos@serpentine.com>
parents:
876
diff
changeset
|
45 # cc = cc1, cc2, ... |
875 | 46 |
47 from email.MIMEMultipart import MIMEMultipart | |
48 from email.MIMEText import MIMEText | |
49 from mercurial import commands | |
50 from mercurial import fancyopts | |
51 from mercurial import hg | |
52 from mercurial import ui | |
53 import os | |
54 import popen2 | |
55 import readline | |
56 import smtplib | |
57 import socket | |
58 import sys | |
59 import tempfile | |
60 import time | |
61 | |
62 def diffstat(patch): | |
63 fd, name = tempfile.mkstemp() | |
64 try: | |
65 p = popen2.Popen3('diffstat -p1 -w79 2>/dev/null > ' + name) | |
66 try: | |
67 for line in patch: print >> p.tochild, line | |
68 p.tochild.close() | |
69 if p.wait(): return | |
70 fp = os.fdopen(fd, 'r') | |
71 stat = [] | |
72 for line in fp: stat.append(line.lstrip()) | |
73 last = stat.pop() | |
74 stat.insert(0, last) | |
75 stat = ''.join(stat) | |
76 if stat.startswith('0 files'): raise ValueError | |
77 return stat | |
78 except: raise | |
79 finally: | |
80 try: os.unlink(name) | |
81 except: pass | |
82 | |
83 def patchbomb(ui, repo, *revs, **opts): | |
84 def prompt(prompt, default = None, rest = ': ', empty_ok = False): | |
876
14cfaaec2e8e
Get patchbomb script to not use MIME attachments.
Bryan O'Sullivan <bos@serpentine.com>
parents:
875
diff
changeset
|
85 if default: prompt += ' [%s]' % default |
14cfaaec2e8e
Get patchbomb script to not use MIME attachments.
Bryan O'Sullivan <bos@serpentine.com>
parents:
875
diff
changeset
|
86 prompt += rest |
14cfaaec2e8e
Get patchbomb script to not use MIME attachments.
Bryan O'Sullivan <bos@serpentine.com>
parents:
875
diff
changeset
|
87 while True: |
875 | 88 r = raw_input(prompt) |
876
14cfaaec2e8e
Get patchbomb script to not use MIME attachments.
Bryan O'Sullivan <bos@serpentine.com>
parents:
875
diff
changeset
|
89 if r: return r |
14cfaaec2e8e
Get patchbomb script to not use MIME attachments.
Bryan O'Sullivan <bos@serpentine.com>
parents:
875
diff
changeset
|
90 if default is not None: return default |
14cfaaec2e8e
Get patchbomb script to not use MIME attachments.
Bryan O'Sullivan <bos@serpentine.com>
parents:
875
diff
changeset
|
91 if empty_ok: return r |
877
25430c523677
Polish patchbomb script.
Bryan O'Sullivan <bos@serpentine.com>
parents:
876
diff
changeset
|
92 ui.warn('Please enter a valid value.\n') |
875 | 93 |
94 def confirm(s): | |
95 if not prompt(s, default = 'y', rest = '? ').lower().startswith('y'): | |
96 raise ValueError | |
97 | |
98 def cdiffstat(summary, patch): | |
99 s = diffstat(patch) | |
100 if s: | |
101 if summary: | |
102 ui.write(summary, '\n') | |
103 ui.write(s, '\n') | |
104 confirm('Does the diffstat above look okay') | |
105 return s | |
106 | |
876
14cfaaec2e8e
Get patchbomb script to not use MIME attachments.
Bryan O'Sullivan <bos@serpentine.com>
parents:
875
diff
changeset
|
107 def makepatch(patch, idx, total): |
875 | 108 desc = [] |
109 node = None | |
1135
e455d91f6259
Variable 'body' was missing in patchbomb script.
Thomas Arendsen Hein <thomas@intevation.de>
parents:
1118
diff
changeset
|
110 body = '' |
875 | 111 for line in patch: |
112 if line.startswith('#'): | |
113 if line.startswith('# Node ID'): node = line.split()[-1] | |
114 continue | |
115 if line.startswith('diff -r'): break | |
116 desc.append(line) | |
117 if not node: raise ValueError | |
1118
63b5f68d8167
patchbomb: eliminate silly complete summary message
mpm@selenic.com
parents:
1032
diff
changeset
|
118 |
63b5f68d8167
patchbomb: eliminate silly complete summary message
mpm@selenic.com
parents:
1032
diff
changeset
|
119 #body = ('\n'.join(desc[1:]).strip() or |
63b5f68d8167
patchbomb: eliminate silly complete summary message
mpm@selenic.com
parents:
1032
diff
changeset
|
120 # 'Patch subject is complete summary.') |
63b5f68d8167
patchbomb: eliminate silly complete summary message
mpm@selenic.com
parents:
1032
diff
changeset
|
121 #body += '\n\n\n' |
63b5f68d8167
patchbomb: eliminate silly complete summary message
mpm@selenic.com
parents:
1032
diff
changeset
|
122 |
877
25430c523677
Polish patchbomb script.
Bryan O'Sullivan <bos@serpentine.com>
parents:
876
diff
changeset
|
123 if opts['diffstat']: |
25430c523677
Polish patchbomb script.
Bryan O'Sullivan <bos@serpentine.com>
parents:
876
diff
changeset
|
124 body += cdiffstat('\n'.join(desc), patch) + '\n\n' |
876
14cfaaec2e8e
Get patchbomb script to not use MIME attachments.
Bryan O'Sullivan <bos@serpentine.com>
parents:
875
diff
changeset
|
125 body += '\n'.join(patch) |
14cfaaec2e8e
Get patchbomb script to not use MIME attachments.
Bryan O'Sullivan <bos@serpentine.com>
parents:
875
diff
changeset
|
126 msg = MIMEText(body) |
875 | 127 subj = '[PATCH %d of %d] %s' % (idx, total, desc[0].strip()) |
128 if subj.endswith('.'): subj = subj[:-1] | |
129 msg['Subject'] = subj | |
876
14cfaaec2e8e
Get patchbomb script to not use MIME attachments.
Bryan O'Sullivan <bos@serpentine.com>
parents:
875
diff
changeset
|
130 msg['X-Mercurial-Node'] = node |
875 | 131 return msg |
132 | |
133 start_time = int(time.time()) | |
134 | |
876
14cfaaec2e8e
Get patchbomb script to not use MIME attachments.
Bryan O'Sullivan <bos@serpentine.com>
parents:
875
diff
changeset
|
135 def genmsgid(id): |
875 | 136 return '<%s.%s@%s>' % (id[:20], start_time, socket.getfqdn()) |
137 | |
138 patches = [] | |
139 | |
140 class exportee: | |
141 def __init__(self, container): | |
142 self.lines = [] | |
143 self.container = container | |
876
14cfaaec2e8e
Get patchbomb script to not use MIME attachments.
Bryan O'Sullivan <bos@serpentine.com>
parents:
875
diff
changeset
|
144 self.name = 'email' |
875 | 145 |
146 def write(self, data): | |
147 self.lines.append(data) | |
148 | |
149 def close(self): | |
150 self.container.append(''.join(self.lines).split('\n')) | |
151 self.lines = [] | |
152 | |
1032
706c590c9060
Get patchbomb working with tip again.
Bryan O'Sullivan <bos@serpentine.com>
parents:
998
diff
changeset
|
153 commands.export(ui, repo, *args, **{'output': exportee(patches), |
706c590c9060
Get patchbomb working with tip again.
Bryan O'Sullivan <bos@serpentine.com>
parents:
998
diff
changeset
|
154 'text': None}) |
875 | 155 |
156 jumbo = [] | |
157 msgs = [] | |
158 | |
159 ui.write('This patch series consists of %d patches.\n\n' % len(patches)) | |
160 | |
161 for p, i in zip(patches, range(len(patches))): | |
162 jumbo.extend(p) | |
876
14cfaaec2e8e
Get patchbomb script to not use MIME attachments.
Bryan O'Sullivan <bos@serpentine.com>
parents:
875
diff
changeset
|
163 msgs.append(makepatch(p, i + 1, len(patches))) |
875 | 164 |
165 ui.write('\nWrite the introductory message for the patch series.\n\n') | |
166 | |
877
25430c523677
Polish patchbomb script.
Bryan O'Sullivan <bos@serpentine.com>
parents:
876
diff
changeset
|
167 sender = (opts['from'] or ui.config('patchbomb', 'from') or |
25430c523677
Polish patchbomb script.
Bryan O'Sullivan <bos@serpentine.com>
parents:
876
diff
changeset
|
168 prompt('From', ui.username())) |
875 | 169 |
170 msg = MIMEMultipart() | |
171 msg['Subject'] = '[PATCH 0 of %d] %s' % ( | |
172 len(patches), | |
877
25430c523677
Polish patchbomb script.
Bryan O'Sullivan <bos@serpentine.com>
parents:
876
diff
changeset
|
173 opts['subject'] or |
875 | 174 prompt('Subject:', rest = ' [PATCH 0 of %d] ' % len(patches))) |
997
458b84a96e1c
Fix handling of addresses in hgrc.
Bryan O'Sullivan <bos@serpentine.com>
parents:
896
diff
changeset
|
175 to = opts['to'] or ui.config('patchbomb', 'to') or prompt('To') |
458b84a96e1c
Fix handling of addresses in hgrc.
Bryan O'Sullivan <bos@serpentine.com>
parents:
896
diff
changeset
|
176 to = [t.strip() for t in to.split(',')] |
877
25430c523677
Polish patchbomb script.
Bryan O'Sullivan <bos@serpentine.com>
parents:
876
diff
changeset
|
177 cc = (opts['cc'] or ui.config('patchbomb', 'cc') or |
997
458b84a96e1c
Fix handling of addresses in hgrc.
Bryan O'Sullivan <bos@serpentine.com>
parents:
896
diff
changeset
|
178 prompt('Cc', default = '')) |
998
c37dd58a444a
contrib/patchbomb: Fix buglet with empty cc list.
Bryan O'Sullivan <bos@serpentine.com>
parents:
997
diff
changeset
|
179 cc = (cc and [c.strip() for c in cc.split(',')]) or [] |
875 | 180 |
181 ui.write('Finish with ^D or a dot on a line by itself.\n\n') | |
182 | |
183 body = [] | |
184 | |
185 while True: | |
186 try: l = raw_input() | |
187 except EOFError: break | |
188 if l == '.': break | |
189 body.append(l) | |
190 | |
191 msg.attach(MIMEText('\n'.join(body) + '\n')) | |
192 | |
193 ui.write('\n') | |
194 | |
195 d = cdiffstat('Final summary:\n', jumbo) | |
196 if d: msg.attach(MIMEText(d)) | |
197 | |
198 msgs.insert(0, msg) | |
199 | |
876
14cfaaec2e8e
Get patchbomb script to not use MIME attachments.
Bryan O'Sullivan <bos@serpentine.com>
parents:
875
diff
changeset
|
200 if not opts['test']: |
14cfaaec2e8e
Get patchbomb script to not use MIME attachments.
Bryan O'Sullivan <bos@serpentine.com>
parents:
875
diff
changeset
|
201 s = smtplib.SMTP() |
14cfaaec2e8e
Get patchbomb script to not use MIME attachments.
Bryan O'Sullivan <bos@serpentine.com>
parents:
875
diff
changeset
|
202 s.connect(host = ui.config('smtp', 'host', 'mail'), |
14cfaaec2e8e
Get patchbomb script to not use MIME attachments.
Bryan O'Sullivan <bos@serpentine.com>
parents:
875
diff
changeset
|
203 port = int(ui.config('smtp', 'port', 25))) |
875 | 204 |
205 parent = None | |
206 tz = time.strftime('%z') | |
207 for m in msgs: | |
208 try: | |
876
14cfaaec2e8e
Get patchbomb script to not use MIME attachments.
Bryan O'Sullivan <bos@serpentine.com>
parents:
875
diff
changeset
|
209 m['Message-Id'] = genmsgid(m['X-Mercurial-Node']) |
875 | 210 except TypeError: |
876
14cfaaec2e8e
Get patchbomb script to not use MIME attachments.
Bryan O'Sullivan <bos@serpentine.com>
parents:
875
diff
changeset
|
211 m['Message-Id'] = genmsgid('patchbomb') |
875 | 212 if parent: |
213 m['In-Reply-To'] = parent | |
876
14cfaaec2e8e
Get patchbomb script to not use MIME attachments.
Bryan O'Sullivan <bos@serpentine.com>
parents:
875
diff
changeset
|
214 else: |
14cfaaec2e8e
Get patchbomb script to not use MIME attachments.
Bryan O'Sullivan <bos@serpentine.com>
parents:
875
diff
changeset
|
215 parent = m['Message-Id'] |
877
25430c523677
Polish patchbomb script.
Bryan O'Sullivan <bos@serpentine.com>
parents:
876
diff
changeset
|
216 m['Date'] = time.strftime('%a, %e %b %Y %T ', time.localtime(start_time)) + tz |
875 | 217 start_time += 1 |
218 m['From'] = sender | |
219 m['To'] = ', '.join(to) | |
220 if cc: m['Cc'] = ', '.join(cc) | |
221 ui.status('Sending ', m['Subject'], ' ...\n') | |
222 if opts['test']: | |
223 fp = os.popen(os.getenv('PAGER', 'more'), 'w') | |
224 fp.write(m.as_string(0)) | |
225 fp.write('\n') | |
226 fp.close() | |
227 else: | |
228 s.sendmail(sender, to + cc, m.as_string(0)) | |
876
14cfaaec2e8e
Get patchbomb script to not use MIME attachments.
Bryan O'Sullivan <bos@serpentine.com>
parents:
875
diff
changeset
|
229 if not opts['test']: |
14cfaaec2e8e
Get patchbomb script to not use MIME attachments.
Bryan O'Sullivan <bos@serpentine.com>
parents:
875
diff
changeset
|
230 s.close() |
875 | 231 |
232 if __name__ == '__main__': | |
233 optspec = [('c', 'cc', [], 'email addresses of copy recipients'), | |
877
25430c523677
Polish patchbomb script.
Bryan O'Sullivan <bos@serpentine.com>
parents:
876
diff
changeset
|
234 ('d', 'diffstat', None, 'add diffstat output to messages'), |
25430c523677
Polish patchbomb script.
Bryan O'Sullivan <bos@serpentine.com>
parents:
876
diff
changeset
|
235 ('f', 'from', '', 'email address of sender'), |
875 | 236 ('n', 'test', None, 'print messages that would be sent'), |
877
25430c523677
Polish patchbomb script.
Bryan O'Sullivan <bos@serpentine.com>
parents:
876
diff
changeset
|
237 ('s', 'subject', '', 'subject of introductory message'), |
875 | 238 ('t', 'to', [], 'email addresses of recipients')] |
239 options = {} | |
240 try: | |
241 args = fancyopts.fancyopts(sys.argv[1:], commands.globalopts + optspec, | |
242 options) | |
243 except fancyopts.getopt.GetoptError, inst: | |
244 u = ui.ui() | |
245 u.warn('error: %s' % inst) | |
246 sys.exit(1) | |
247 | |
248 u = ui.ui(options["verbose"], options["debug"], options["quiet"], | |
249 not options["noninteractive"]) | |
250 repo = hg.repository(ui = u) | |
251 | |
252 patchbomb(u, repo, *args, **options) |