[PATCH V2] run-tests: '--time' option provide more details to linux users

timeless at gmail.com timeless at gmail.com
Mon Jun 9 22:30:23 CDT 2014


> As our tests exceute in child process, this patch includes os.times()

Spell check? Execute 
Child processes (plural) 

> module in replace of time.time() module to provide addition info like

I think you mean function and not module, thus uses instead of includes. 

Additional 

> children's user time and children's system time along with real time

Child's or each child's 

Rewritten:
The user and system time spent by each child process 

> taken by a process.
‎
> There is one limitation of this patch.It can only work for linux users and not

Space after period. 
Capitalize Linux 

> for windows users because in windows, os.times() module work only for parent

And Windows. 

> process.

What does this actually mean? Will the tests break on Windows? 

You need to have a Windows test showing the output with this flag. 


More information about the Mercurial-devel mailing list