[PATCH 3 of 5] pager: skip uisetup if chg is detected

Sean Farley sean at farley.io
Mon Mar 14 16:53:52 EDT 2016


Jun Wu <quark at fb.com> writes:

> On 03/14/2016 08:37 PM, Sean Farley wrote:
>> Would this be reason enough to put pager in core so we wouldn't need
>> different implementations?
>
> The reason that chg needs a different implementation is the pager must be
> executed by the client. Otherwise it's likely to have trouble with
> "terminal foreground job", Ctrl+Z etc.
>
> To be more precise, the pager is currently using subprocess.Popen to create
> process at server side. chg needs it to be "remotely" executed at client
> side.
>
> I think Yuya has plans to unify the implementations somehow. See
> https://www.mercurial-scm.org/pipermail/mercurial-devel/2016-March/081393.html

Yes, ok, this is more of what I was hoping (thanks Yuya for the ideas).


More information about the Mercurial-devel mailing list