00:04 <asp_> @headius, thanks for your work on #5863. I'll test this against the original code that alerted me to this issue & let you know how I go. I also have an open PR (#5866) which you may or may not wish to use - just rebased it to see if it passes now

00:05 <headius[m]> oh nice I did not see that

00:05 <headius[m]> this is basically the example you provided in test form?

00:05 <asp_> yeah, I modified it to prevent it from hanging (hanging specs are bad)

00:05 <headius[m]> it's properly using the mutex for the sleep now, with a bit of a tweak to allow JVM thread status to be translated to Ruby status

00:06 <headius[m]> those fixes are merged in now

00:06 <asp_> Yeah, sounds like letting the JVM take care of these tricky locking issues is a good way to go

00:07 <headius[m]> it's doing a similar sequence to what I was doing before, but it's obviously sleeping against the proper lock

00:07 <headius[m]> this is a great one to get in, thank you for chasing down the hanging example

00:08 <asp_> Happy to be of service. Thank you for all your work on this most excellent project!

00:26 <asp_> headius: looks good - my tests are now passing using a build of JRuby from master 👍️

00:27 <headius[m]> Excellent!

00:29 <asp_> Looks like travis is having some issues though: https://travis-ci.org/jruby/jruby/jobs/603378702#L568

00:49 <headius[m]> Yeah, that is my fault. I just released that library and it takes a while to propagate

00:49 <headius[m]> I just get tired of sitting and waiting to push the build update

00:49 <headius[m]> Should work fine if we restart in half an hour or so

00:51 <headius[m]> Master is broken too at the moment

00:51 <headius[m]> Oh, or not

00:51 <headius[m]> I guess it was far enough down in the queue that the library propagated

07:10 <headius[m]> asp: all merged in!

07:55 <travis-ci> jruby/jruby (master:8a75d31 by Charles Oliver Nutter): The build passed. https://travis-ci.org/jruby/jruby/builds/603444391 [201 min 2 sec]

08:04 <travis-ci> jruby/jruby (revert-5915-dir-open:dcf3905 by Charles Oliver Nutter): The build failed. https://travis-ci.org/jruby/jruby/builds/603446899 [192 min 45 sec]

08:24 <travis-ci> jruby/jruby (master:76ceb12 by Charles Oliver Nutter): The build passed. https://travis-ci.org/jruby/jruby/builds/603449452 [203 min 12 sec]

10:50 <asp_> <headius[m] "asp: all merged in!"> Great!