patchbomb: drop internal option for pbranch extension (API)
I want to move _getpatches() to _getpatchmsgs() to make sure each patch text
is tied with the corresponding revision number. This helps adding templater
support.
IIRC, the pbranch extension doesn't work with the recent Mercurial versions,
so the removal of this option wouldn't hurt.
--- a/hgext/patchbomb.py Sat Feb 25 18:16:41 2017 +0900
+++ b/hgext/patchbomb.py Sat Feb 25 18:25:17 2017 +0900
@@ -514,14 +514,12 @@
mbox = opts.get('mbox')
outgoing = opts.get('outgoing')
rev = opts.get('rev')
- # internal option used by pbranches
- patches = opts.get('patches')
if not (opts.get('test') or mbox):
# really sending
mail.validateconfig(ui)
- if not (revs or rev or outgoing or bundle or patches):
+ if not (revs or rev or outgoing or bundle):
raise error.Abort(_('specify at least one changeset with -r or -o'))
if outgoing and bundle:
@@ -593,10 +591,7 @@
ui.config('patchbomb', 'from') or
prompt(ui, 'From', ui.username()))
- if patches:
- msgs = _getpatchmsgs(repo, sender, patches, opts.get('patchnames'),
- **opts)
- elif bundle:
+ if bundle:
bundledata = _getbundle(repo, dest, **opts)
bundleopts = opts.copy()
bundleopts.pop('bundle', None) # already processed