This is an archive of the discontinued Mercurial Phabricator instance.

push: respect pushvars
ClosedPublic

Authored by quark on Aug 16 2017, 7:20 PM.
Tags
None
Subscribers
None

Details

Reviewers
ryanmce
Summary

The pushvars feature was recently added in core. This patch adds
corresponding support in remotenames.

Depends on D423.

Diff Detail

Lint
Lint Skipped
Unit
Unit Tests Skipped

Event Timeline

quark created this revision.Aug 16 2017, 7:20 PM
ryanmce edited edge metadata.Aug 17 2017, 7:57 AM

Why wasn't this needed when this was an extension in fb-hgext?

It seems that we shouldn't need every other extension to be aware of pushvars. This seems like a step backwards to me.

ryanmce requested changes to this revision.Aug 17 2017, 7:57 AM

Back to you for discussion

This revision now requires changes to proceed.Aug 17 2017, 7:57 AM
quark added a comment.EditedAug 17 2017, 10:32 AM

The pushvars extension was doing wrapcommand('push', ...). Now the pushvars logic is in core Mecurial push command. remotenames bypasses the entire core Mercurial push command implementation when --to is provided.

ryanmce accepted this revision.Aug 17 2017, 2:51 PM

Yuck. Oh well.

This revision is now accepted and ready to land.Aug 17 2017, 2:51 PM

This is now in our hotfixes while we wait for the core changes to get in to unblock this stuff.

quark updated this revision to Diff 1176.Aug 22 2017, 11:55 PM
quark closed this revision.Sep 10 2017, 7:08 PM