-
Notifications
You must be signed in to change notification settings - Fork 59
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
Downgrade rainbow dependency #180
Conversation
Rainbow in latest version seems to have become a C extension which doesn't correctly compile in certain environments. Lets downgrade it for now.
@gnufied i will try this branch for couple of day, and let you know. thanks. |
How did it go @iffyuva ? |
@gnufied all good |
I think the latest release of rainbow may be fixing this, just FYI. |
There doesn't seem to be any C in the codebase ... I think there was just a bad 2.2.x release. See their changelog. A version bump would be appreciated to get rid of the |
ack, will push a update tonight. sorry about it. |
Their release of 2.2.2 appears to be borked still. I am sometimes forced to use ruby bundled with distributions and I am getting following error while trying to install that gem with default ruby installation on Ubuntu 16.04:
It isn't looking great unfortunately.. |
Filed a issue with them - ku1ik/rainbow#69 Will push a fix as soon as they fix it. |
Have you tried updating Rubygems? It seems that people have other installation issues on Ruby 2.3+ that are resolved by updating rubygems, although that doesn't bode well. I've never seen a gem that requires a certain version of rubygems... |
I installed rubygems via |
Can you guys please do something about this. I can't upgrade to rubocop 0.50 because of this:
|
@swrobel I have removed rainbow as a dependency and moved this gem to colorize. Let me know if with version 1.5.5 things look good. |
@gnufied thank you! 1.5.5 fixes the issue. |
Rainbow in latest version seems to have become a
C extension which doesn't correctly compile in
certain environments. Lets downgrade it for now.
CC @iffyuva