D2101: infinitepush: remove wrapping around bundle2._addpartsfromopts()

2018-03-30 Thread pulkit (Pulkit Goyal)
This revision was automatically updated to reflect the committed changes.
Closed by commit rHGe702ca152b33: infinitepush: remove wrapping around 
bundle2._addpartsfromopts() (authored by pulkit, committed by ).

REPOSITORY
  rHG Mercurial

CHANGES SINCE LAST UPDATE
  https://phab.mercurial-scm.org/D2101?vs=5362=7416

REVISION DETAIL
  https://phab.mercurial-scm.org/D2101

AFFECTED FILES
  hgext/infinitepush/__init__.py

CHANGE DETAILS

diff --git a/hgext/infinitepush/__init__.py b/hgext/infinitepush/__init__.py
--- a/hgext/infinitepush/__init__.py
+++ b/hgext/infinitepush/__init__.py
@@ -165,9 +165,6 @@
 configitem('infinitepush', 'reponame',
 default='',
 )
-configitem('infinitepush', 'bundle-stream',
-default=False,
-)
 configitem('scratchbranch', 'storepath',
 default='',
 )
@@ -360,7 +357,6 @@
 extensions.wrapcommand(commands.table, 'update', _update)
 
 extensions.wrapfunction(discovery, 'checkheads', _checkheads)
-extensions.wrapfunction(bundle2, '_addpartsfromopts', _addpartsfromopts)
 
 wireproto.wirepeer.listkeyspatterns = listkeyspatterns
 
@@ -427,18 +423,6 @@
 return
 return orig(pushop)
 
-def _addpartsfromopts(orig, ui, repo, bundler, *args, **kwargs):
-""" adds a stream level part to bundle2 storing whether this is an
-infinitepush bundle or not
-This functionality is hidden behind a config option:
-
-[infinitepush]
-bundle-stream = True
-"""
-if ui.configbool('infinitepush', 'bundle-stream', False):
-bundler.addparam('infinitepush', True)
-return orig(ui, repo, bundler, *args, **kwargs)
-
 def wireprotolistkeyspatterns(repo, proto, namespace, patterns):
 patterns = wireproto.decodelist(patterns)
 d = repo.listkeys(encoding.tolocal(namespace), patterns).iteritems()
@@ -1255,14 +1239,6 @@
 if bundle:
 bundle.close()
 
-@bundle2.b2streamparamhandler('infinitepush')
-def processinfinitepush(unbundler, param, value):
-""" process the bundle2 stream level parameter containing whether this push
-is an infinitepush or not. """
-if value and unbundler.ui.configbool('infinitepush',
- 'bundle-stream', False):
-pass
-
 @bundle2.parthandler(scratchbranchparttype,
  ('bookmark', 'bookprevnode' 'create', 'force',
   'pushbackbookmarks', 'cgversion'))



To: pulkit, #hg-reviewers, indygreg
Cc: mercurial-devel
___
Mercurial-devel mailing list
Mercurial-devel@mercurial-scm.org
https://www.mercurial-scm.org/mailman/listinfo/mercurial-devel


D2101: infinitepush: remove wrapping around bundle2._addpartsfromopts()

2018-02-09 Thread pulkit (Pulkit Goyal)
pulkit created this revision.
Herald added a subscriber: mercurial-devel.
Herald added a reviewer: hg-reviewers.

REVISION SUMMARY
  This wrapping around bundle2._addpartsfromopts() was added by me while hacking
  on an alternate way to mark a push as infinitepush or not. However the 
wrapping
  is wrong as the push command does not go through the code path which was 
wrapped
  on the client side.

REPOSITORY
  rHG Mercurial

REVISION DETAIL
  https://phab.mercurial-scm.org/D2101

AFFECTED FILES
  hgext/infinitepush/__init__.py

CHANGE DETAILS

diff --git a/hgext/infinitepush/__init__.py b/hgext/infinitepush/__init__.py
--- a/hgext/infinitepush/__init__.py
+++ b/hgext/infinitepush/__init__.py
@@ -165,9 +165,6 @@
 configitem('infinitepush', 'reponame',
 default='',
 )
-configitem('infinitepush', 'bundle-stream',
-default=False,
-)
 configitem('scratchbranch', 'storepath',
 default='',
 )
@@ -360,7 +357,6 @@
 extensions.wrapcommand(commands.table, 'update', _update)
 
 extensions.wrapfunction(discovery, 'checkheads', _checkheads)
-extensions.wrapfunction(bundle2, '_addpartsfromopts', _addpartsfromopts)
 
 wireproto.wirepeer.listkeyspatterns = listkeyspatterns
 
@@ -427,18 +423,6 @@
 return
 return orig(pushop)
 
-def _addpartsfromopts(orig, ui, repo, bundler, *args, **kwargs):
-""" adds a stream level part to bundle2 storing whether this is an
-infinitepush bundle or not
-This functionality is hidden behind a config option:
-
-[infinitepush]
-bundle-stream = True
-"""
-if ui.configbool('infinitepush', 'bundle-stream', False):
-bundler.addparam('infinitepush', True)
-return orig(ui, repo, bundler, *args, **kwargs)
-
 def wireprotolistkeyspatterns(repo, proto, namespace, patterns):
 patterns = wireproto.decodelist(patterns)
 d = repo.listkeys(encoding.tolocal(namespace), patterns).iteritems()
@@ -1255,14 +1239,6 @@
 if bundle:
 bundle.close()
 
-@bundle2.b2streamparamhandler('infinitepush')
-def processinfinitepush(unbundler, param, value):
-""" process the bundle2 stream level parameter containing whether this push
-is an infinitepush or not. """
-if value and unbundler.ui.configbool('infinitepush',
- 'bundle-stream', False):
-pass
-
 @bundle2.parthandler(scratchbranchparttype,
  ('bookmark', 'bookprevnode' 'create', 'force',
   'pushbackbookmarks', 'cgversion'))



To: pulkit, #hg-reviewers
Cc: mercurial-devel
___
Mercurial-devel mailing list
Mercurial-devel@mercurial-scm.org
https://www.mercurial-scm.org/mailman/listinfo/mercurial-devel