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

SVG URLs not rendering in PDF #250

Open
casperdcl opened this issue Nov 4, 2024 · 5 comments
Open

SVG URLs not rendering in PDF #250

casperdcl opened this issue Nov 4, 2024 · 5 comments

Comments

@casperdcl
Copy link

casperdcl commented Nov 4, 2024

Source: https://github.com/casperdcl/test/tree/refs/heads/pandoc-test

![](https://img.shields.io/badge/test-pandoc-green.svg)

Expected: pandoc -s -t pdf should work fine

Got: PDF art[ie]fact downloadable from https://github.com/casperdcl/test/actions/runs/11659385994

image

@jgm
Copy link
Member

jgm commented Nov 4, 2024

This works fine for me when I try it locally.
What version of pandoc are you using? If you're running it in an action, is rsvg-convert available?

@tarleb
Copy link
Member

tarleb commented Nov 4, 2024

I can confirm this behavior for pandoc/latex:latest-alpine, but it works fine for pandoc/latex:latest-ubuntu. Graphic libraries seem to have problems when compiled against musl. See, e.g., reports of very elongated images produced by graphviz on Alpine.

Please use a Ubuntu-base image, there's nothing we can do about this on our end.

@tarleb tarleb closed this as completed Nov 4, 2024
@tarleb
Copy link
Member

tarleb commented Nov 4, 2024

On second thought, maybe we should consider to make Ubuntu-based images the default, so pandoc/core:latest would become an alias for pandoc/core:latest-ubuntu instead of pandoc/core:latest-alpine.

@tarleb tarleb reopened this Nov 4, 2024
@daamien
Copy link
Collaborator

daamien commented Nov 6, 2024

On second thought, maybe we should consider to make Ubuntu-based images the default, so pandoc/core:latest would become an alias for pandoc/core:latest-ubuntu instead of pandoc/core:latest-alpine.

Seems like a huge change to me because we would also have to point the latest tag to ubuntu on the other images too, otherwise it would be kind of a mess.... Overall that could break thousands of CI pipelines and other workflows....

@tarleb
Copy link
Member

tarleb commented Nov 7, 2024

Indeed, it would induce a lot of breakage, so this step would have to be well-planned, well-documented, and widely announced.

There's another reason why I'm tempted to do that in the future: we don't have LaTeX images for Alpine on arm64. I see that as a serious problem, as the default template should be the most general, and it should not come with a list of platform limitations and asterisks.

I've also been toying the idea to move from Ubuntu to Debian at some point, simply because Debian seems a bit more stable and closer to the traditional free software spirit. If we do that, then that might be a good opportunity to change the default as well.

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

No branches or pull requests

4 participants