( )⚙ D2993 bundle2: add 'source' atrribute to bundleoperation class

This is an archive of the discontinued Mercurial Phabricator instance.

Differential D2993

bundle2: add 'source' atrribute to bundleoperation class
ClosedPublic

Authored by pulkit on Mar 31 2018, 6:23 AM.

Details

Reviewers
indygreg
Group Reviewers
hg-reviewers
Commits
rHG7e906d8a825f: bundle2: add 'source' atrribute to bundleoperation class
Summary

This will help us in easily finding out which command leads to the current
operation without hacking on the transaction.

Diff Detail

Repository
rHG Mercurial
Lint
Lint Skipped
Unit
Unit Tests Skipped

Event Timeline

pulkit created this revision.Mar 31 2018, 6:23 AM
indygreg accepted this revision.Mar 31 2018, 1:14 PM
This revision is now accepted and ready to land.Mar 31 2018, 1:14 PM
Closed by commit rHG7e906d8a825f: bundle2: add 'source' atrribute to bundleoperation class (authored by pulkit). · Explain WhyMar 31 2018, 1:18 PM
This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

PathPackages
Mmercurial/bundle2.py (3 lines)

Diff 7460

mercurial/bundle2.py

The object has very little content now it will ultimately contain: The object has very little content now it will ultimately contain:
* an access to the repo the bundle is applied to, * an access to the repo the bundle is applied to,
* a ui object, * a ui object,
* a way to retrieve a transaction to add changes to the repo, * a way to retrieve a transaction to add changes to the repo,
* a way to record the result of processing each part, * a way to record the result of processing each part,
* a way to construct a bundle response when applicable. * a way to construct a bundle response when applicable.
""" """
def __init__(self, repo, transactiongetter, captureoutput=True): def __init__(self, repo, transactiongetter, captureoutput=True, source=''):
self.repo = repo self.repo = repo
self.ui = repo.ui self.ui = repo.ui
self.records = unbundlerecords() self.records = unbundlerecords()
self.reply = None self.reply = None
self.captureoutput = captureoutput self.captureoutput = captureoutput
self.hookargs = {} self.hookargs = {}
self._gettransaction = transactiongetter self._gettransaction = transactiongetter
# carries value that can modify part behavior # carries value that can modify part behavior
self.modes = {} self.modes = {}
self.source = source
def gettransaction(self): def gettransaction(self):
transaction = self._gettransaction() transaction = self._gettransaction()
if self.hookargs: if self.hookargs:
# the ones added to the transaction supercede those added # the ones added to the transaction supercede those added
# to the operation. # to the operation.
self.hookargs.update(transaction.hookargs) self.hookargs.update(transaction.hookargs)