Skip to content

HTTPS clone URL

Subversion checkout URL

You can clone with
or
.
Download ZIP

Loading…

Remove optional rendering engines (RedCloth, maruku) from the dependency list #57

Closed
bmaland opened this Issue · 5 comments

4 participants

@bmaland

I don't think it's a good idea to make optional rendering engines forced requirements. The current main issue with Jekyll's Ruby 1.9 compability is the RedCloth 4.1.0 requirement (4.1.9 is the first RedCloth version that works with Ruby 1.9). Thus, people that doesn't even want to use RedCloth for their markup currently can't install Jekyll because of this (Ruby 1.9 users, that is). I don't see any good reasons for keeping them as dependencies, we could just rescue LoadErrors and print installation instructions instead.

(I'd be happy to implement the necessary changes, if they would be accepted into the mainline)

@qrush

I don't fully understand...so how will textile/markdown get rendered if we're not using the external libraries? I think it would be most unwise to implement an engine for either inside of Jekyll.

I'm up for any sort of work related to fixing this problem, it's definitely a huge pain point.

@bmaland

Maybe I didn't get my idea across properly (I'm not a native speaker) - I'll try again. :)

Sure, I think these libraries should be used to render the pages, but they aren't really requirements for using Jekyll. If I want to use markdown, and for some reason prefer rdiscount, I don't need Jekyll to install RedCloth and maruku as dependencies as I won't even use these other engines.

So instead of having these as dependencies for installing the gem, they could be run-time dependencies. RDiscount actually works the way I suggest right now. If you chose RDiscount in _config.yml without having it installed, you will simply get a LoadError telling you how to install it. That's how RedCloth and maruku should be treated as well, imo.

@qrush

Ahh, I see now. +1 then :)

@tvo

See also #128

@mojombo
Owner

Maruku is now the only processor dependency installed by default. Closed by de8bd48.

Other processors will be lazy-loaded when necessary (and prompt the
user to install them when necessary).

@mgriffin mgriffin referenced this issue from a commit
@mojombo mojombo Maruku is now the only processor dependency installed by default. Clo…
…ses #57.

Other processors will be lazy-loaded when necessary (and prompt the
user to install them when necessary).
de8bd48
This issue was closed.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Something went wrong with that request. Please try again.