[PATCH 4 of 4 V2] resolve: wrap implementation in a ui.tempconfig context

Siddharth Agarwal sid0 at fb.com
Tue Mar 22 13:30:54 EDT 2016


On 3/22/16 10:22, Martin von Zweigbergk wrote:
> I like patches 1&4, but I don't see how they're related to 2&3. Could
> 4 be done before the move instead so it's not held hostage by patch 2?
>
> I was also wondering if 1 config option is always enough, but we can
> of course always add support for passing a list of options later if
> there even are such cases (I could not find any after spending ~2 min
> looking).

Not without massive indentation diffs, which were part of my goal with 2 
and 3 anyway.


More information about the Mercurial-devel mailing list