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

Change the way RMagick is required by the base class. #123

Closed
wants to merge 1 commit into from
Closed

Change the way RMagick is required by the base class. #123

wants to merge 1 commit into from

Conversation

trytriangles
Copy link

The old form is deprecated, and throws the following warning whenever Gruff is used:

[DEPRECATION] requiring "RMagick" is deprecated. Use "rmagick" instead

@topfunky
Copy link
Owner

Hmm...it fails on JRuby under TravisCI. Does it use a different version of RMagick there?

@donv
Copy link
Collaborator

donv commented May 30, 2015

Hi @ryanplant-au !

Thanks for the PR! We went with #127 instead since it supported JRuby as well.

@donv donv closed this May 30, 2015
@donv donv modified the milestone: 0.6.0 May 30, 2015
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

Successfully merging this pull request may close these issues.

None yet

3 participants