[PATCH 1 of 3] ui: introduce util.system() wrapper to make sure ui.fout is used

Pierre-Yves David pierre-yves.david at ens-lyon.org
Sat Nov 8 04:27:47 CST 2014



On 11/08/2014 05:42 AM, Yuya Nishihara wrote:
> # HG changeset patch
> # User Yuya Nishihara <yuya at tcha.org>
> # Date 1415419062 -32400
> #      Sat Nov 08 12:57:42 2014 +0900
> # Node ID d83746a91dc410f60b45916c2e55dadde4238b42
> # Parent  2d54aa5397cdb1c697673ba10b7618d5ac25c69e
> ui: introduce util.system() wrapper to make sure ui.fout is used

The idea looks good, but can I get you to add some docstring the new 
method so that people understand its purpose?

As far as I understand the lack of it makes thg command server usage to 
break in multiple occasion. Should this be target on stable?

Finally, Do we have a plan to force people to use ui.system instead of 
util.system?


-- 
Pierre-Yves David


More information about the Mercurial-devel mailing list