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
Don't add rake dependency if bundler has already added it #84
Comments
I think later versions of bundler add that dependency, and so methadone no longer needs to. Methadone should only be adding an open dependency, though: https://github.com/davetron5000/methadone/blob/master/bin/methadone#L82 This was new in 1.8.0, released last month: https://github.com/davetron5000/methadone/releases/tag/v1.8.0 |
OK, I'm using 1.8.0 of Methadone and Bundler is at 1.6.4. Should methadone 1.8.0 not have added this rule?
Sorry this is all fairly new/foreign to me so I'm trying to make heads or tails of what's what. |
A fresh install of methadone on bundler 1.5.3 adds a rake dependency without the version specifier, so it's possible its bundler. I think the problem is that older versions of bundler didn't add a rake specifier at all, and now they do. Since methadone uses bundler under the covers to bootstrap the gem, and also just inserts a rake dependency to cover its bases, I think that's the problem. So, I think the ultimate bug here is that methadone should ony add the rake dependency if it's not in the gemspec already. That way, for older versions of bundler, it would add it, but newer it would see it an not. Gonna change the title of this issue to match and mark it as a bug. Even thought it's minor, there's no need to cause confusion—methadone is supposed to help you :) |
Thanks Dave. Been going through your free ibook too. Very helpful. Thanks for your hard work, it's appreciated. Will be picking up a printed copy of the PragPub book as a thank you! |
This has been fixed (not sure when), but methadone won't result in rake being in the gemspec twice. |
Methadone makes the following gemspec:
Which results in this msg:
Taking out the extra restriction about rake:
spec.add_development_dependency "rake", "~> 10.0"
makes the msg go away. Why is this restriction in the gemspec?The text was updated successfully, but these errors were encountered: