This is an archive of the discontinued Mercurial Phabricator instance.

infinitepush: don't wrap bundle2.processparts while calling `hg unbundle`
ClosedPublic

Authored by pulkit on Mar 27 2018, 8:38 AM.

Details

Summary

This patch adds dirty logic to check whether we are processing hg unbundle
instead of an hg incoming to prevent the wrapping of bundle2.processparts
function.

Diff Detail

Repository
rHG Mercurial
Lint
Automatic diff as part of commit; lint not applicable.
Unit
Automatic diff as part of commit; unit tests not applicable.

Event Timeline

pulkit created this revision.Mar 27 2018, 8:38 AM
indygreg requested changes to this revision.Mar 30 2018, 1:59 PM
indygreg added a subscriber: indygreg.

Can the op argument be used instead?

Also, given the hackiness of this, I'd feel much better if there were test coverage for hg unbundle continuing to work with this extension enabled.

This revision now requires changes to proceed.Mar 30 2018, 1:59 PM
indygreg accepted this revision.Mar 30 2018, 2:05 PM

The next patch adds test coverage. So I think this is fine. A follow-up to make this less hacky (if possible) would be appreciated, but isn't required.

This revision is now accepted and ready to land.Mar 30 2018, 2:05 PM
This revision was automatically updated to reflect the committed changes.