seattlerb / debride

Analyze code for potentially uncalled / dead methods, now with auto-removal.
https://www.zenspider.com/projects/debride.html
720 stars 19 forks source link

undefined method `expand_dirs_to_files' for Debride:Class #2

Closed take-five closed 9 years ago

take-five commented 9 years ago

Hi,

Tried to test my app with debride but got following error:

$ debride app/
/usr/local/rvm/gems/ruby-2.1.2/gems/debride-1.0.0/lib/debride.rb:24:in `run': undefined method `expand_dirs_to_files' for Debride:Class (NoMethodError)
    from /usr/local/rvm/gems/ruby-2.1.2/gems/debride-1.0.0/bin/debride:5:in `<top (required)>'
    from /usr/local/rvm/gems/ruby-2.1.2/bin/debride:23:in `load'
    from /usr/local/rvm/gems/ruby-2.1.2/bin/debride:23:in `<main>'
    from /usr/local/rvm/gems/ruby-2.1.2/bin/ruby_executable_hooks:15:in `eval'
    from /usr/local/rvm/gems/ruby-2.1.2/bin/ruby_executable_hooks:15:in `<main>'
imageaid commented 9 years ago

Got the same error as @take-five but on Ruby 2.2.1

zenspider commented 9 years ago

Dependency error. Update your sexp_processor gem.

On Mar 11, 2015, at 06:37, Alexey Mihaylov notifications@github.com wrote:

Hi,

Tried to test my app with debride but got following error:

$ debride app/ /usr/local/rvm/gems/ruby-2.1.2/gems/debride-1.0.0/lib/debride.rb:24:in run': undefined methodexpand_dirs_to_files' for Debride:Class (NoMethodError) from /usr/local/rvm/gems/ruby-2.1.2/gems/debride-1.0.0/bin/debride:5:in <top (required)>' from /usr/local/rvm/gems/ruby-2.1.2/bin/debride:23:inload' from /usr/local/rvm/gems/ruby-2.1.2/bin/debride:23:in <main>' from /usr/local/rvm/gems/ruby-2.1.2/bin/ruby_executable_hooks:15:ineval' from /usr/local/rvm/gems/ruby-2.1.2/bin/ruby_executable_hooks:15:in `

' — Reply to this email directly or view it on GitHub.

imageaid commented 9 years ago

Thank you @zenspider ... that did the trick for me. Appreciate it!

take-five commented 9 years ago

I believe, gems should declare proper dependencies. Why not set correct version in .gemspec file?

zenspider commented 9 years ago

@take-five: as said before, it's an error.