-
Notifications
You must be signed in to change notification settings - Fork 32
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
Method missing: full_require_paths on macOS system Ruby #73
Comments
... and bundler monkey-patches it in. I can fix this |
@segiddins @orta any plans to release this fix on CLAide and then update the Danger requirement? We've just bumped into this very same issue. Happy to help if I can, although it sounds just like a mundane release step. Thanks ❤️ |
I thought this would have come out with one of the CocoaPods beta releases? @dantoml did you notice this ever coming up? Does it require a change to the CHANGELOG to trigger it being noticed? |
@orta yeah during super release you only see the changelog, and the tests all pass on my machine & CI so assumed it was good |
ps. to update rubygems:
|
@orta I can do a release with a fix for this whenever tho |
Sure thing, I'm not very free this evening, but I can make a PR adding the changelog in anticipation of a release ( or you're welcome to beat me to it ) |
@wilrnh that doesn't work on 10.11
|
you need sudo in that case:
edit: I was using RVM, which does not require sudo to update gems |
@wilrnh That is defeating the point of this issue, as it should work out of the box with a default RubyGems installation on OS X. |
Def I didn't mean it as fix, just a possible workaround in case you're between a rock and a hard place. |
Added the CHANGELOG - #75 |
So, this is the stack trace. Specifically when calling
danger
, nobundle exec
.$ gem -v
=2.0.14.1
It looks like
full_require_paths
was added in Rubygems 2.2 - which isn't installed by default http://blog.rubygems.org/2013/12/26/2.2.0-released.htmlhttps://github.com/CocoaPods/CLAide/blob/master/lib/claide/command/plugin_manager.rb#L59
This only happens when a plugin crashes, which is why we've probably not seen it in CocoaPods yet, probably cause there's real software engineering going on over there 💃
/cc @samdmarshall
The text was updated successfully, but these errors were encountered: