Re: std.process.execute performance without Config.inheritFDs

2017-07-03 Thread bauss via Digitalmars-d 

On Monday, 3 July 2017 at 20:34:00 UTC, Ali Çehreli wrote:

On 07/03/2017 07:53 AM, Vladimir Panteleev wrote:

> That code is pretty old. std.process was changed to use poll
when
> available over a year ago:
>
> https://github.com/dlang/phobos/pull/4114
>
> Update your D installation?

Jonathan is hailing from Weka. :) As evidenced from Johan's 
recent messages, they're in the process of updating their ldc.


Ali


But then what's the point of this thread?

Re: std.process.execute performance without Config.inheritFDs

2017-07-03 Thread Ali Çehreli via Digitalmars-d 

On 07/03/2017 07:53 AM, Vladimir Panteleev wrote:

> That code is pretty old. std.process was changed to use poll when
> available over a year ago:
>
> https://github.com/dlang/phobos/pull/4114
>
> Update your D installation?

Jonathan is hailing from Weka. :) As evidenced from Johan's recent 
messages, they're in the process of updating their ldc.


Ali

Re: std.process.execute performance without Config.inheritFDs

2017-07-03 Thread Vladimir Panteleev via Digitalmars-d 

On Monday, 3 July 2017 at 14:48:38 UTC, Jonathan Shamir wrote:

if (!(config & Config.inheritFDs))
{
import core.sys.posix.sys.resource;
rlimit r;
getrlimit(RLIMIT_NOFILE, );
foreach (i; 3 .. cast(int) r.rlim_cur) close(i);
}


That code is pretty old. std.process was changed to use poll when 
available over a year ago:


https://github.com/dlang/phobos/pull/4114

Update your D installation?