-
-
Notifications
You must be signed in to change notification settings - Fork 24
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
pandoc should be an optional, not required, dependency #24
Comments
I'm reluctant to remove |
The main reason pandoc isn't required by the Python package is because it can't be. There isn't a pip package for it. If pandoc were a Python package, it would probably be a strict dependency of nbconvert. Since pandoc is required for most things nbconvert does (HTML output being the main exception), it makes sense for nbconvert to depend on it in conda, since it can. If conda had a "recommended dependencies" facility like apt, where a 'default' install would get what's needed for most things to work (pandoc), but users could opt-in to a "just the basics" install, we could leave out nbconvert, analogous to |
It's been years now, but w have the convention of a |
Sure, I can warm #47 back up. There appear to be <30 feedstock that directly reference |
#47 is ready for review |
I think that
pandoc
should be an optional dependency ofnbconvert
, not a required one.Per the
nbconvert
docs:The
nbconvert
README mentionspandoc
only in the "development install" instructions and does not claim it as a required dependency.For my use-case I only care about converting to HTML and I don't want
pandoc
to be automatically installed because it adds significant file-size bloat. This is especially a problem when moving frompip
toconda
becausepip install nbconvert
didn't installpandoc
.Conda has a currently-undocumented feature for marking optional dependencies as
run_constrained
: conda/conda#1696. Perhaps this mechanism could be used to express the relationship betweennbconvert
andpandoc
without forcing the latter to be installed?Perhaps my use-case is uncommon, but I wanted to open an issue to start discussion because I currently need to perform an ugly workaround to remove
pandoc
after installation.The text was updated successfully, but these errors were encountered: