comparison contrib/patchbomb @ 1118:63b5f68d8167

patchbomb: eliminate silly complete summary message No longer makes sense without attachments
author mpm@selenic.com
date Sat, 27 Aug 2005 23:10:49 -0700
parents 706c590c9060
children e455d91f6259
comparison
equal deleted inserted replaced
1117:30ab5b8ee8ec 1118:63b5f68d8167
112 if line.startswith('# Node ID'): node = line.split()[-1] 112 if line.startswith('# Node ID'): node = line.split()[-1]
113 continue 113 continue
114 if line.startswith('diff -r'): break 114 if line.startswith('diff -r'): break
115 desc.append(line) 115 desc.append(line)
116 if not node: raise ValueError 116 if not node: raise ValueError
117 body = ('\n'.join(desc[1:]).strip() or 117
118 'Patch subject is complete summary.') 118 #body = ('\n'.join(desc[1:]).strip() or
119 body += '\n\n\n' 119 # 'Patch subject is complete summary.')
120 #body += '\n\n\n'
121
120 if opts['diffstat']: 122 if opts['diffstat']:
121 body += cdiffstat('\n'.join(desc), patch) + '\n\n' 123 body += cdiffstat('\n'.join(desc), patch) + '\n\n'
122 body += '\n'.join(patch) 124 body += '\n'.join(patch)
123 msg = MIMEText(body) 125 msg = MIMEText(body)
124 subj = '[PATCH %d of %d] %s' % (idx, total, desc[0].strip()) 126 subj = '[PATCH %d of %d] %s' % (idx, total, desc[0].strip())