ruby-changes:35904
From: normal <ko1@a...>
Date: Fri, 17 Oct 2014 03:12:05 +0900 (JST)
Subject: [ruby-changes:35904] normal:r47985 (trunk): Revert r47971, r47972
normal 2014-10-17 03:11:49 +0900 (Fri, 17 Oct 2014) New Revision: 47985 http://svn.ruby-lang.org/cgi-bin/viewvc.cgi?view=revision&revision=47985 Log: Revert r47971, r47972 [ruby-core:65764] The current implementation currently delivers signals to self immediately and synchronously, so we do not need explicit, potentially-confusing sleeps to wait for signal delivery. * test/-ext-/bug_reporter/test_bug_reporter.rb (test_bug_reporter_add): revert r47972 * test/ruby/test_rubyoptions.rb (test_segv_test): revert r47971 [ruby-core:65764] Modified files: trunk/ChangeLog trunk/test/-ext-/bug_reporter/test_bug_reporter.rb trunk/test/ruby/test_rubyoptions.rb Index: ChangeLog =================================================================== --- ChangeLog (revision 47984) +++ ChangeLog (revision 47985) @@ -1,3 +1,10 @@ https://github.com/ruby/ruby/blob/trunk/ChangeLog#L1 +Fri Oct 17 03:05:08 2014 Eric Wong <e@8...> + + * test/-ext-/bug_reporter/test_bug_reporter.rb + (test_bug_reporter_add): revert r47972 + * test/ruby/test_rubyoptions.rb (test_segv_test): revert r47971 + [ruby-core:65764] + Thu Oct 16 23:17:40 2014 Nobuyoshi Nakada <nobu@r...> * signal.c (rb_sigaltstack_size): double default size to get rid Index: test/ruby/test_rubyoptions.rb =================================================================== --- test/ruby/test_rubyoptions.rb (revision 47984) +++ test/ruby/test_rubyoptions.rb (revision 47985) @@ -569,7 +569,7 @@ class TestRubyOptions < Test::Unit::Test https://github.com/ruby/ruby/blob/trunk/test/ruby/test_rubyoptions.rb#L569 end def test_segv_test - assert_segv(["--disable-gems", "-e", "Process.kill :SEGV, $$; sleep"]) + assert_segv(["--disable-gems", "-e", "Process.kill :SEGV, $$"]) end def test_segv_loaded_features Index: test/-ext-/bug_reporter/test_bug_reporter.rb =================================================================== --- test/-ext-/bug_reporter/test_bug_reporter.rb (revision 47984) +++ test/-ext-/bug_reporter/test_bug_reporter.rb (revision 47985) @@ -16,7 +16,7 @@ class TestBugReporter < Test::Unit::Test https://github.com/ruby/ruby/blob/trunk/test/-ext-/bug_reporter/test_bug_reporter.rb#L16 args = ["--disable-gems", "-r-test-/bug_reporter/bug_reporter", "-C", tmpdir] - stdin = "register_sample_bug_reporter(12345); Process.kill :SEGV, $$; sleep" + stdin = "register_sample_bug_reporter(12345); Process.kill :SEGV, $$" _, stderr, status = EnvUtil.invoke_ruby(args, stdin, false, true) stderr.force_encoding("ASCII-8BIT") assert_pattern_list(expected_stderr, stderr) -- ML: ruby-changes@q... Info: http://www.atdot.net/~ko1/quickml/