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

Added Mutex.new statement to prevent deadlocking error #28

Closed
bootstraponline opened this issue Jan 17, 2015 · 3 comments
Closed

Added Mutex.new statement to prevent deadlocking error #28

bootstraponline opened this issue Jan 17, 2015 · 3 comments

Comments

@bootstraponline
Copy link
Member

From @saranyan on June 16, 2011 18:9

Hi, in ruby 1.9, there were errors when I tried to use the gem.
internal:prelude:8:in `lock': deadlock; recursive locking (ThreadError)
I traced it to base.rb statement - @mailbox_mutex.synchronize do
This does not make sense to me as it should be a mutex variable. I just made this into mutex variable and the errors went away. Thought this might be useful to others.

thanks!
great Gem. very useful. thanks :)

Copied from original issue: nu7hatch/gmail/pull/28

@bootstraponline
Copy link
Member Author

From @nu7hatch on August 17, 2011 18:38

deadlocking went away, but this solution breaks thread-safety. I think for this line i will just revert thread safety additions, and move work on it to a separate branch. Anyway, thanks for the help!

@bootstraponline
Copy link
Member Author

From @nu7hatch on August 17, 2011 18:47

Ok, so i'm closing this pull req without merging. Keep an eye on this issue nu7hatch/gmail#34

@bootstraponline
Copy link
Member Author

From @zenzei on July 5, 2012 18:36

I was having the same problem and this workaround help me in a ruby 1.9.3p125 (I understand is not thread safe, but I could live without it for a while).

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

No branches or pull requests

1 participant