Skip to content

Can you please install the package "librsvg2-bin"? #78

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

Closed
mgeier opened this issue Oct 23, 2018 · 4 comments · Fixed by #84
Closed

Can you please install the package "librsvg2-bin"? #78

mgeier opened this issue Oct 23, 2018 · 4 comments · Fixed by #84
Labels
Improvement Minor improvement to code
Milestone

Comments

@mgeier
Copy link
Contributor

mgeier commented Oct 23, 2018

This would allow auto-converting SVG images to PDF images with the Sphinx extension https://github.com/missinglinkelectronics/sphinxcontrib-svg2pdfconverter.

There is an old issue about converting SVG to PDF which would be properly solved by this: #905.

Alternatively, Inkscape could be installed, but librsvg2-bin is much more lightweight, even more so since its dependency librsvg2-2 seems to be already installed.

Using the built-in Sphinx extension sphinx.ext.imgconverter (http://www.sphinx-doc.org/en/master/usage/extensions/imgconverter.html) would be and alternative (but much inferior) solution. This uses convert from ImageMagick (see #3979 and #70), which rasterizes vector graphics and produces bitmaps.

Expected Result

When the PyPI package sphinxcontrib-svg2pdfconverter is installed and 'sphinxcontrib.rsvgconverter' is listed in the extensions in conf.py, SVG images should be automatically be converted to PDF images to be included in the LaTeX/PDF output.

Actual Result

Running Sphinx v1.8.1
...
building [latex]: all documents
...
WARNING: RSVG converter command 'rsvg-convert' cannot be run. Check the rsvg_converter_bin setting
...
@agjohnson
Copy link
Contributor

agjohnson commented Nov 10, 2018

Moving this to our docker build images, this is feature addition there.

Edit: thanks GitHub issue transfer! 🎉

@agjohnson agjohnson transferred this issue from readthedocs/readthedocs.org Nov 10, 2018
@agjohnson agjohnson added the Improvement Minor improvement to code label Nov 10, 2018
@agjohnson agjohnson added this to the 4.1 milestone Nov 10, 2018
@agjohnson
Copy link
Contributor

Also, the change is pretty easy, feel free to open a PR against our master branch

mgeier added a commit to mgeier/readthedocs-docker-images that referenced this issue Nov 12, 2018
@mgeier
Copy link
Contributor Author

mgeier commented Nov 12, 2018

Thanks @agjohnson, I've created #79, I hope that's the right way.

@mgeier
Copy link
Contributor Author

mgeier commented Nov 12, 2018

FYI, the issue transfer disabled two issue links in the original issue description.

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

Successfully merging a pull request may close this issue.

2 participants