##// END OF EJS Templates
patchbomb: consistently use opts.get
Christian Ebert -
r7095:0ed11838 default
parent child Browse files
Show More
@@ -159,7 +159,7 b' def patchbomb(ui, repo, *revs, **opts):'
159 if not node:
159 if not node:
160 raise ValueError
160 raise ValueError
161
161
162 if opts['attach']:
162 if opts.get('attach'):
163 body = ('\n'.join(desc[1:]).strip() or
163 body = ('\n'.join(desc[1:]).strip() or
164 'Patch subject is complete summary.')
164 'Patch subject is complete summary.')
165 body += '\n\n\n'
165 body += '\n\n\n'
@@ -190,7 +190,7 b' def patchbomb(ui, repo, *revs, **opts):'
190 else:
190 else:
191 patchname = cmdutil.make_filename(repo, '%b.patch', binnode)
191 patchname = cmdutil.make_filename(repo, '%b.patch', binnode)
192 disposition = 'inline'
192 disposition = 'inline'
193 if opts['attach']:
193 if opts.get('attach'):
194 disposition = 'attachment'
194 disposition = 'attachment'
195 p['Content-Disposition'] = disposition + '; filename=' + patchname
195 p['Content-Disposition'] = disposition + '; filename=' + patchname
196 msg.attach(p)
196 msg.attach(p)
General Comments 0
You need to be logged in to leave comments. Login now