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

FIX #87: Add tectonic to extra #228

Merged
merged 1 commit into from
May 14, 2024
Merged

FIX #87: Add tectonic to extra #228

merged 1 commit into from
May 14, 2024

Conversation

daamien
Copy link
Collaborator

@daamien daamien commented May 4, 2024

Tectonic is a promising pdf engine.

It's ability to automatically download latex support files on-the-fly could solve a lot of missing package x or missing font y problems.

Since this is still experimental, I think we should add it to the extras image and maybe move it uptream into the latex image in the forthcoming month if it is stable and useful...

Closes #87

@daamien daamien self-assigned this May 4, 2024
Copy link
Member

@tarleb tarleb left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM!

@tarleb
Copy link
Member

tarleb commented May 5, 2024

I'm having second thoughts. This currently assumes a x86-64 architecture, which will lead to issues if we try to support other architectures like arm.

@daamien
Copy link
Collaborator Author

daamien commented May 5, 2024

Yes

there's an apk package but no deb package for Ubuntu

Maybe we can use conda ?

https://tectonic-typesetting.github.io/book/latest/installation

@tarleb
Copy link
Member

tarleb commented May 5, 2024

Adding another package manager seems like a rather heavy solution. Would it be possible to write an install command that works with multiple architectures?

Another possibility could be to ask the tectonic devs to publish a package on CTAN, that would make the installation trivial.

@daamien
Copy link
Collaborator Author

daamien commented May 6, 2024

@tarleb I found this solution:

3c1b85a#diff-86bf175f2e6e05c2f1307ec16481ae84c75506c1261e550fa8212e380d25eadbR182

not pretty but it works

let me know if that's ok for you

@tarleb
Copy link
Member

tarleb commented May 6, 2024

Neat, looks good. How much space does that add to the image?

@daamien
Copy link
Collaborator Author

daamien commented May 6, 2024

The tectonic binary is 55MB all by itself :)

root@1d11f0afdaad:/data# ls -lh /usr/local/bin/tectonic                                                              
-rwxr-xr-x 1 1001 127 55M Feb  5 00:56 /usr/local/bin/tectonic

@tarleb
Copy link
Member

tarleb commented May 6, 2024

That's like half a pandoc, so not too bad. 😄

There seems to be a PATH issues on Alpine that needs fixing, but I have no objections to merging this 👍

@daamien
Copy link
Collaborator Author

daamien commented May 7, 2024

Somehow the binary-drop installation wouldn't work on Alpine. I switched back to the apk install method.

https://github.com/pandoc/dockerfiles/pull/228/files#diff-86bf175f2e6e05c2f1307ec16481ae84c75506c1261e550fa8212e380d25eadbR185

@daamien daamien merged commit 59ed827 into master May 14, 2024
5 checks passed
@tarleb tarleb deleted the 87_tectonic branch May 15, 2024 05:16
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.

Tectonic
2 participants