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

Install poppler-utils (to get pdftoppm) #154

Closed
wants to merge 2 commits into from

Conversation

mgeier
Copy link
Contributor

@mgeier mgeier commented Oct 21, 2020

Can you please install the package poppler-utils?

This provides the program pdftoppm for converting PDF files to bitmaps (like PNG, JPEG, TIFF, ...).

This conversion could theoretically also be done with convert (from ImageMagick, which is already installed), but this doesn't work, see #155.

Apart from that, my experiments so far have shown that pdftoppm produces better results.

@humitos
Copy link
Member

humitos commented Apr 1, 2021

Hi and thanks for your contribution!

I suggest you to subscribe to this PR readthedocs/readthedocs.org#8065 that implements a new config in .readthedocs.yaml that allow users to install packages via APT. Once it gets merged and deployed you will be able to install these packages by yourself.

Since we are allowing users to install these dependencies, I don't think this PR will be required anymore. I'm closing it. Let me know if anything.

@humitos humitos closed this Apr 1, 2021
@mgeier
Copy link
Contributor Author

mgeier commented May 18, 2021

Thanks @humitos, I've just tried it with the new apt_packages feature and it works great!

However, there will still be a certain number of basic packages installed by default, right?

I think it would still be helpful to install the package poppler-utils by default, just like convert from ImageMagick is installed by default.

Conversion between common image formats should work out of the box, IMHO.

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

Successfully merging this pull request may close these issues.

2 participants