Bug #960: SIGINT signal is not handled properly on OpenSolaris

• Target version changed from 1.9.1 Release Candidate to 1.9.1 RC2

Hi,

> Following test failed.. (http://ruby.peppercode.com/log/ruby-1.9.1/last.txt )
>
> test_should_propagate_signaled(TestBeginEndBlock) [/usr/local/src/ruby-1.9.1-rc1/test/ruby/test_beginendblock.rb:96]
>
> It seems that a ruby loop doesn't get interrupted by 'Ctrl-C' on OpenSolaris.


I think the following patch will fix this issue.  Could you try it?

Index: process.c
===================================================================
--- process.c	(revision 21266)
+++ process.c	(working copy)
@@ -2736,12 +2736,8 @@
 }


-#if defined(sun)
-#define signal(a,b) sigset(a,b)
-#else
-# if defined(POSIX_SIGNAL)
-#  define signal(a,b) posix_signal(a,b)
-# endif
+#if defined(POSIX_SIGNAL)
+# define signal(a,b) posix_signal(a,b)
 #endif

 void

-- 
Yusuke ENDOH <[e-mail]>

Hi Yusuke,

Your patch indeed solves the problem.

It fixed two other process related test failures as well. Thanks a lot!

daesan


> I think the following patch will fix this issue.  Could you try it?

> Index: process.c
> ===================================================================
> --- process.c	(revision 21266)
> +++ process.c	(working copy)
> @@ -2736,12 +2736,8 @@
>  }


> -#if defined(sun)
> -#define signal(a,b) sigset(a,b)
> -#else
> -# if defined(POSIX_SIGNAL)
> -#  define signal(a,b) posix_signal(a,b)
> -# endif
> +#if defined(POSIX_SIGNAL)
> +# define signal(a,b) posix_signal(a,b)
>  #endif

>  void

• Status changed from Open to Closed
• % Done changed from 0 to 100

Applied in changeset r21275.

Hi Yusuke,

Can you apply this patch to 1.9.1 branch as well? Thanks!

daesan

• Assignee set to yugui (Yuki Sonoda)