[PATCH 2 of 4 V2] resolve: move implementation into cmdutil

Siddharth Agarwal sid at less-broken.com
Tue Mar 22 13:18:02 EDT 2016


On 3/22/16 10:16, Martin von Zweigbergk via Mercurial-devel wrote:
> Why is it imported here? I've seen this kind of thing done in a few
> other places, but can you remind me when it's needed/useful? Is it
> some import cycle? If it is, then which direction is correct, for
> cmdutil to depend on merge or for merge to depend on cmdutil?

Hmm, I think there's a cycle involving three or four different modules. 
Will need to dig in to find out more.

>
> I think I saw someone send patches for resolving a cycle involving
> mergestate. Do you know if that's related?

Yeah, that's the real fix here.


More information about the Mercurial-devel mailing list