ruby-changes:30929
From: nobu <ko1@a...>
Date: Sat, 21 Sep 2013 20:11:28 +0900 (JST)
Subject: [ruby-changes:30929] nobu:r43008 (trunk): process.c: remove cloexec setting
nobu 2013-09-21 20:11:10 +0900 (Sat, 21 Sep 2013) New Revision: 43008 http://svn.ruby-lang.org/cgi-bin/viewvc.cgi?view=rev&revision=43008 Log: process.c: remove cloexec setting * process.c (rb_fork_internal): remove cloexec setting on pipes created by rb_cloexec_pipe. patch by normalperson (Eric Wong) at [ruby-core:56523]. [Bug #8769] Modified files: trunk/ChangeLog trunk/process.c Index: ChangeLog =================================================================== --- ChangeLog (revision 43007) +++ ChangeLog (revision 43008) @@ -1,3 +1,9 @@ https://github.com/ruby/ruby/blob/trunk/ChangeLog#L1 +Sat Sep 21 20:11:06 2013 Nobuyoshi Nakada <nobu@r...> + + * process.c (rb_fork_internal): remove cloexec setting on pipes + created by rb_cloexec_pipe. patch by normalperson (Eric Wong) at + [ruby-core:56523]. [Bug #8769] + Sat Sep 21 01:04:25 2013 Zachary Scott <e@z...> * lib/benchmark.rb: [DOC] grammar of Benchmark#bm [Bug #8888] Index: process.c =================================================================== --- process.c (revision 43007) +++ process.c (revision 43008) @@ -3389,10 +3389,6 @@ rb_fork_internal(int *status, int (*chfu https://github.com/ruby/ruby/blob/trunk/process.c#L3389 } else { if (pipe_nocrash(ep, fds)) return -1; - if (fcntl(ep[1], F_SETFD, FD_CLOEXEC)) { - preserving_errno((close(ep[0]), close(ep[1]))); - return -1; - } pid = retry_fork(status, ep, chfunc_is_async_signal_safe); if (pid < 0) return pid; -- ML: ruby-changes@q... Info: http://www.atdot.net/~ko1/quickml/