[前][次][番号順一覧][スレッド一覧]

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/

[前][次][番号順一覧][スレッド一覧]