programming-journal / programming

21 stars 14 forks source link

Add a GitHub Actions setup #48

Closed smarr closed 3 years ago

smarr commented 3 years ago

I was perhaps a little too optimistic, though, it turns out it's really hard to make the example compile.

The simplest setups are in the first two commits.

Unfortunately, there's an issue with the tabularx package, which doesn't like the tabular in the example.

Trying a Ubuntu 18.04 setup fails because we need a newer hyperxmp.

Ubuntu 20.04 fixes that issue.

Though, now I am back to the other issue with the texlive docker image from the first two commits: the Fira font doesn't work with XeLaTex.

My Latex foo isn't strong enough for this.

So, here a draft PR to see whether someone else might know what's needed to make this work.

And a final note: while the use of plain Ubuntu is slow, it documents which packages are actually needed.

krono commented 3 years ago

I'm very Inclined to just merge, but could you please give https://github.com/dante-ev/latex-action a try? (If you have already, sorry…)

smarr commented 3 years ago

Doesn't make a difference to the FireSans-Book not being "found" issue.

https://github.com/smarr/programming/runs/2726111983?check_suite_focus=true#step:6:226

From what I read, this could perhaps be an unsupported font type (webfont file instead of normal font) or something like that. The fire package itself is installed, I tried that before.

krono commented 3 years ago

eeb8af2 should fix the Font issue, apparently xelatex got a bit fussy in 2021.

smarr commented 3 years ago

Indeed, that fixes it. (sorry, for the distraction...)

Any preference over plain Ubuntu setup (6min run time) or the Dante texlive Docker image-based one? (4min run time)

krono commented 3 years ago

It's ok :)

The Dante one is maintained by TeX people, so I'm in favor of that one.

krono commented 3 years ago

(must have been a change to xelatex in 2021)

smarr commented 3 years ago

Pushed the Dante one, and removed the draft marker. So, could be merged.

krono commented 3 years ago

:shipit: