jasonwebster / tumblargh

MIT License
128 stars 20 forks source link

Moved alias line in order to avoid 'method not found' errors. #20

Open telamon opened 10 years ago

telamon commented 10 years ago

Hi! I got this error everytime i tried running middleman

$ bundle exec middleman
/.../tumblargh/lib/middleman/extensions/tumblargh.rb:30:in `alias_method': undefined method `find_resource_by_destination_path' for class `Middleman::Sitemap::Store' (NameError)

Moving the alias line to the bottom solved this problem.

jasonwebster commented 10 years ago

But that will create the alias pointing to the method we're overriding. Obviously something in the internals of middleman have changed. Can you post your Gemfile.lock please?

telamon commented 10 years ago

Ah that explains it! I was wondering why noone else had voiced any concerns against this issue. I accidentally got it running by moving that alias line and did't think twice about why it was there to begin with.

You can see my Gemfile.lock here: https://gist.github.com/telamon/af0d26984a6a2b70b0d7

jasonwebster commented 10 years ago

Thanks! I'll definitely have to spend some time figuring out what has changed here and will update as soon as I reasonably can.

In the mean time, if you wanted to get up and running right away, you can use an earlier version of middleman. I know ~> 3.1.0 would be safe.

On Sat, May 3, 2014 at 4:07 AM, Tony Ivanov notifications@github.comwrote:

Ah that explains it! I was wondering why noone else had voiced any concerns against this issue. I accidentally got it running by moving that alias line and did't think twice about why it was there to begin with.

You can see my Gemfile.lock here: https://gist.github.com/telamon/af0d26984a6a2b70b0d7

— Reply to this email directly or view it on GitHubhttps://github.com/jasonwebster/tumblargh/pull/20#issuecomment-42102333 .

Jason Webster CTO | www.metalabdesign.com