-
Notifications
You must be signed in to change notification settings - Fork 334
Description
Hello, today I was using jekyll-import version 0.1.0.beta3 gem to migrate my blog from wordpress to jekyll using directly the connection to MySQL server and not from the exported .xml file. As I was doing this, I encounter that it was missing 2 runtime dependencies: sequel and mysql2.
I don't know if this was on purpose or it was mistake, I'm just pointing it out. these where the errors:
ruby -rubygems -e 'require "jekyll/jekyll-import/wordpress";
quote> JekyllImport::WordPress.process({:host => "127.0.0.1", :dbname => "blah", :user => "root", :pass => ""})'
/Users/albertogg/.rbenv/versions/2.0.0-p247/lib/ruby/2.0.0/rubygems/core_ext/kernel_require.rb:106:in `require': cannot load such file -- sequel (LoadError)After installing sequel version 3.42 (it is the same version you guys have as development dependency) I got the mysql2 error.
ruby -rubygems -e 'require "jekyll/jekyll-import/wordpress";
JekyllImport::WordPress.process({:host => "127.0.0.1", :dbname => "blah", :user => "root", :pass => ""})'
Could not require 'htmlentities', so the :clean_entities option is now disabled.
/Users/albertogg/.rbenv/versions/2.0.0-p247/lib/ruby/2.0.0/rubygems/core_ext/kernel_require.rb:106:in `require': LoadError: cannot load such file -- mysql2 (Sequel::AdapterNotFound)After installing mysql2 all errors where gone, and the import was done correctly. Also, from what I saw jekyll-import/lib/jekyll/jekyll-import/wordpress.rb is the only one using mysql2 gem and it's not a development dependency. I can submit a pull request to fix this, I just don't know what's your opinion about all of this.
Thank you.