Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fiber#alive? regression in 1.7.11 #1789

Closed
judofyr opened this issue Jul 4, 2014 · 2 comments
Closed

Fiber#alive? regression in 1.7.11 #1789

judofyr opened this issue Jul 4, 2014 · 2 comments
Milestone

Comments

@judofyr
Copy link

judofyr commented Jul 4, 2014

require 'fiber'

fiber = Fiber.new do
  Fiber.yield
end

fiber.resume while fiber.alive?

This works fine in MRI and JRuby 1.7.10. Output in 1.7.11:

FiberError: dead fiber called
  resume at org/jruby/ext/fiber/ThreadFiber.java:57
  (root) at bug.rb:7

This breaks template streaming in Rails. See slim-template/slim#514.

@splattael
Copy link
Contributor

This regression prevents some ActionPack tests from passing.

dmke added a commit to slim-template/slim that referenced this issue Dec 29, 2014
As of 

- jruby/jruby#1243 and
- jruby/jruby#1789,

there might be a regression in JRuby 1.7.x, while JRuby 9000 works
just fine. 

(Side note: please don't branch off this commit as long there is
an Work-In-Progress tag. I will force push some minor changes
when fiddleing with Travis...)
dmke added a commit to slim-template/slim that referenced this issue Dec 30, 2014
As of

- jruby/jruby#1243 and
- jruby/jruby#1789,

there might be a regression in JRuby 1.7.x, while JRuby 9000 works
just fine.
dmke added a commit to slim-template/slim that referenced this issue Jan 4, 2015
As of

- jruby/jruby#1243 and
- jruby/jruby#1789,

there might be a regression in JRuby 1.7.x, while JRuby 9000 works
just fine.
dmke added a commit to slim-template/slim that referenced this issue Jan 30, 2015
As of

- jruby/jruby#1243 and
- jruby/jruby#1789,

there might be a regression in JRuby 1.7.x, while JRuby 9000 works
just fine.
@enebo enebo added this to the Won't Fix milestone May 17, 2017
@enebo
Copy link
Member

enebo commented May 17, 2017

This seems to be workign in 9.x. 1.7.x is EOL and this will not get fixed. Marking WONTFIX

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

4 participants