Skip to content
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.

Sign up for GitHub

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

Jump to bottom

keep getting uninitialized @@mtimes on jekyll versions above 3.1.6 #12

Closed
jericbryledy opened this issue Sep 1, 2016 · 5 comments
Closed

keep getting uninitialized @@mtimes on jekyll versions above 3.1.6 #12

jericbryledy opened this issue Sep 1, 2016 · 5 comments

Comments

@jericbryledy
Copy link

jekyll 3.2.1 | Error: uninitialized class variable @@mtimes in Jekyll::JekyllMinimagick::GeneratedImageFile
@ellioseven
Copy link

+1

@jericbryledy
Copy link
Author

I think this is what's causing the problem: jekyll/jekyll@7a93389

@jp26jp
Copy link

jp26jp commented Oct 4, 2016

+1

@franzos
Copy link
Contributor

franzos commented Oct 5, 2016

Here's how you can temporarily fix the issue:

  1. Open jekyll-minimagick.rb
  2. Find "@@mtimes[path] = mtime" around line 43
  3. Replace with "self.class.mtimes[path] = mtime"
  4. Run Jekyll and enjoy!

If you have no idea where to find the file, use "gem env" to figure out the "GEM PATHS". On MacOS, the full path will look something like this: "/Users/[USERNAME]/.rvm/gems/ruby-2.1.1/gems/jekyll-minimagick-0.0.3/lib"

PS: I've posted a gist with the fix here.

@franzos franzos mentioned this issue Oct 5, 2016
Merged
jericbryledy added a commit to jericbryledy/jerikyll that referenced this issue Oct 25, 2016
@jericbryledy
just updated gemfile.lock
6bc2213 
note:
	must manually update jekyll-minimagick
	follow issue: zroger/jekyll-minimagick#12
	pull request: zroger/jekyll-minimagick#13
zroger added a commit that referenced this issue Nov 27, 2016
@zroger
Merge pull request #13 from franzos/patch-1
f92b521 
Fixed Issue #12
@zroger
Copy link
Owner

zroger commented Nov 27, 2016

Fixed by #13.

@zroger zroger closed this as completed Nov 27, 2016
@iceboundflame iceboundflame mentioned this issue Dec 2, 2016
Open
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
5 participants
@zroger @franzos @ellioseven @jericbryledy @jp26jp