search

moshen / jekyll-asset_bundler

A small plugin for Jekyll to easily bundle and compress site assets
MIT License
101 stars 21 forks source link

Error with jekyll 2.0.2 #22

Closed jbleuzen closed 10 years ago

jbleuzen commented 10 years ago

Wanted to upgrade to latest jekyll just for fun, encountered a bug and it seams that deep_merge is the culprit on line 106.

Here is the following error I got : 

...
  Liquid Exception: undefined method `deep_merge' for #<Hash:0x007fd05a730488> in _includes/head.html, included in _layouts/post.html
/Users/johan/.rbenv/versions/2.0.0-p247/lib/ruby/gems/2.0.0/gems/jekyll-2.0.2/lib/jekyll/tags/include.rb:114:in `rescue in render': undefined method `deep_merge' for #<Hash:0x007fd05a730488> (Jekyll::Tags::IncludeTagError)
        from /Users/johan/.rbenv/versions/2.0.0-p247/lib/ruby/gems/2.0.0/gems/jekyll-2
....

Don't bother to solve that quickly, I reverted to a previous version of jekyll :)

daneden commented 10 years ago

+1 on this error.

moshen commented 10 years ago

Thanks guys, Jekyll 2 compatibility is the first thing on my list when I get a moment.

ryanhornberger commented 10 years ago

+1 great work so far

johnykov commented 10 years ago

awesome, I've already impaled myself on this while calling "jekyll build/serve"

jekyll 2.0.3 | Error: undefined method `deep_merge' for #Hash:0x007fa8f98723b0

moshen commented 10 years ago

master is fixed, tested with Jekyll 2.0.3.

jbleuzen commented 10 years ago

Perfect, thank you !

iamsalnikov commented 10 years ago

Jekyll 2.1.1 Ruby 2.1.2

Same error (jekyll 2.1.1 | Error: undefined method 'deep_merge')

moshen commented 10 years ago

Have you tried the current master? Not using the deep_merge util anymore.

iamsalnikov commented 10 years ago

@moshen, I did not use current master. But when I downgrade my ruby to 2.0 all works fine.