-
Notifications
You must be signed in to change notification settings - Fork 67
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
rbtree collides with rmagick on aws linux #641
Comments
Does this happen with |
I cannot reproduce the segfault. When the file doesn't exist, I get:
When the file exists, the script exits with 0. I am on Ruby 2.7.1, rmagick 4.1.2, rbtree3 0.6.0 |
Just reproduced it using a brand new elastic beanstalk passenger with ruby 2.6 running on Amazon Linux 64 bit. 2.12.2 with ruby 2.6.6, rmagick 4.1.2, rbtree3 0.6.0 I just created brand new environment to validate. Two things:
|
Thanks, so does this happen when using https://rubygems.org/gems/rbtree ? |
It does. Happens with both of them. |
Thanks for verifying this. So I'm reading your repro script but I don't see I'm probably missing something but I don't see where |
It appears that rbtree3 was your fork of rbtree if I'm not mistaken. It is also a dependency that of course renders airbrake-ruby unusable to us which is why I mentioned it here. You could generate the same error by require 'airbrake-ruby' instead of require 'rbtree' It does appear that if you require 'rmagick' and reference a class from it before requiring 'rbtree' that you can in fact make it work. However, since airbrake monkey patches several core rails infrastructure classes, the only real workaround I have found is to put a Happy to move this over to rbtree and rmagick as well if you like, just thought you might want to be aware, and see if you had any ideas |
The issue doesn't seem to be related specifically to I am not an |
I get a segmentation fault when attempting to use rmagick if and only if airbrake has loaded. I have traced it down to the rbtree3 gem. To reproduce:
Not sure about how the extension is compiled here, but seems to be problematic.
The text was updated successfully, but these errors were encountered: