diff options
-rw-r--r-- | ChangeLog | 10 | ||||
-rw-r--r-- | process.c | 2 | ||||
-rw-r--r-- | test/ruby/test_process.rb | 21 |
3 files changed, 32 insertions, 1 deletions
@@ -1,3 +1,13 @@ +Wed Jul 10 14:18:59 2013 Nobuyoshi Nakada <nobu@ruby-lang.org> + + * process.c (fork_daemon): kill the other threads all and abandon the + kept mutexes. + +Wed Jul 10 14:14:44 2013 Nobuyoshi Nakada <nobu@ruby-lang.org> + + * process.c (fork_daemon): kill the other threads all and abandon the + kept mutexes. + Wed Jul 10 11:35:36 2013 NAKAMURA Usaku <usa@ruby-lang.org> * test/net/http/test_http.rb (TestNetHTTP_v1_2#test_get, @@ -5742,7 +5742,7 @@ rb_daemon(int nochdir, int noclose) #define fork_daemon() \ switch (rb_fork_ruby(NULL)) { \ case -1: return -1; \ - case 0: break; \ + case 0: rb_thread_atfork(); break; \ default: _exit(EXIT_SUCCESS); \ } diff --git a/test/ruby/test_process.rb b/test/ruby/test_process.rb index 9a01b60acd..bfe58457d7 100644 --- a/test/ruby/test_process.rb +++ b/test/ruby/test_process.rb @@ -1472,6 +1472,15 @@ class TestProcess < Test::Unit::TestCase assert_equal("ok?\n", data) end + def test_daemon_pid + cpid, dpid = IO.popen("-", "r+") do |f| + break f.pid, Integer(f.read) if f + Process.daemon(false, true) + puts $$ + end + assert_not_equal(cpid, dpid) + end + if File.directory?("/proc/self/task") && /netbsd[a-z]*[1-6]/ !~ RUBY_PLATFORM def test_daemon_no_threads pid, data = IO.popen("-", "r+") do |f| @@ -1483,6 +1492,18 @@ class TestProcess < Test::Unit::TestCase assert_equal(2, data.size, bug4920) assert_not_include(data.map(&:to_i), pid) end + else + def test_daemon_no_threads + data = Timeout.timeout(3) do + IO.popen("-") do |f| + break f.readlines.map(&:chomp) if f + th = Thread.start {sleep 3} + Process.daemon(true, true) + puts Thread.list.size, th.status.inspect + end + end + assert_equal(["1", "false"], data) + end end end |