Re: [PATCH 3/3] Add a sleep in alarm test in current Ruby.

[Eric Wong <normalperson@yhbt.net>]

Hleb Valoshka <375gnu@gmail.com> wrote:
> Without that sleep alarms will be 0. Looks like Ruby VM issue.

Thanks, your other two patches look good and I've pushed them.

Can you elaborate on this issue?

I would like to see a bug report/analysis to Ruby upstream
(and or EGLIBC) maintainers.
I assume this only happens on GNU/kFreeBSD?

> --- a/test/test_posix_mq.rb
> +++ b/test/test_posix_mq.rb
> @@ -127,6 +127,7 @@ class Test_POSIX_MQ < Test::Unit::TestCase
>      a = nil
>      assert_raises(Errno::EINTR) { a = @mq.receive }
>      elapsed = Time.now - t0
> +    sleep 0.01 if RUBY_VERSION >= '1.9'
>      assert_nil a
>      assert elapsed >= interval, elapsed.inspect
>      assert elapsed < 1.10, elapsed.inspect

Fwiw, I'm not sure if this is a good test, either.  Since I
wrote this code, I've come to believe normal API methods should
not raise Errno::EINTR.  I've changed a few of my other libraries
(sleepy_penguin, kgio) to avoid EINTR but missed this one.