On 13/02/2017 18:04, Bryan O'Sullivan wrote:
On Mon, Feb 13, 2017 at 9:29 AM, Simon Farnsworth <simon...@fb.com <mailto:simon...@fb.com>> wrote: + with ui.timeblockedsection('extdiff'): + ui.system(cmdline, cwd=tmproot) Why not simply instrument ui.system directly, and leave its callers untouched?
I want the caller to tell me what tag to apply to this blocking reason, so that I can account for the blocking in hook.py (where it's running a non-interactive process) and sshpeer.py (where it's network blocking as we talk to a server) differently to extdiff.py and filemerge.py (where it's an interactive process).
I could push the instrumentation into ui.system, conditional on being passed a tag (with a catch-all for "not tagged") if people think that would be more useful?
-- Simon Farnsworth _______________________________________________ Mercurial-devel mailing list Mercurial-devel@mercurial-scm.org https://www.mercurial-scm.org/mailman/listinfo/mercurial-devel