patchbomb: drop internal option for pbranch extension (API)
authorYuya Nishihara <yuya@tcha.org>
Sat, 25 Feb 2017 18:25:17 +0900
changeset 31184 e3ab7c717129
parent 31183 8018b90f8307
child 31185 bbb5d2aa0bf0
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.
hgext/patchbomb.py
--- 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