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.

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

Missing py_modules in setup.py? #2

Open
silverhook opened this issue May 10, 2014 · 8 comments
Open

Missing py_modules in setup.py? #2

silverhook opened this issue May 10, 2014 · 8 comments
Labels

Comments

@silverhook
Copy link

I can’t seem to make this work under Pelican …I am wondering if the following line is perhaps missing in setup.py:

py_modules=['mdx_highlight']

@ribalba
Copy link
Owner

ribalba commented May 12, 2014

Hey, I don't think we need that dependence? How did you come to that?

@silverhook
Copy link
Author

I don’t mean that you depend on it, but from what I understand (and I could be very wrong) the point of that variable is to let others know how this module is called.

At least comparing your setup.py to the one from mdx_del_ins that’s how I understand it.

It could very well also be that Pelican assumes that the names of all Python Markdown extensions are prefixed by mdx_.

@domenkozar
Copy link

@silverhook
Copy link
Author

Friendly ping?

@ribalba
Copy link
Owner

ribalba commented Dec 26, 2014

Hey, sorry totally lost track of this. Is on the todo list now. Sorry about that.

@silverhook
Copy link
Author

That’s totally fine. Glad to hear you’re still working on this plugin 😸

@ribalba
Copy link
Owner

ribalba commented Dec 29, 2014

Hey, so I took a look at this. You can use markdown.highlight with Pelican by adding the following lines to your pelicanconf.py:

from MarkdownHighlight.highlight import HighlightExtension
MD_EXTENSIONS = [HighlightExtension(), ]

and this seams to work fine in my environment.

Markdown==2.5.2
MarkdownHighlight==0.1.3
pelican==3.5.0

does this help you? Thx @iElectric for the hint.

@silverhook
Copy link
Author

Seems a bit hackish, but it works – thanks 😸

If you at some point manage to get it working without having to import it manually, it would be even neater.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

3 participants