Skip to content

Commit

Permalink
chore(tooling): Relax requirement on updated preview for the sake of CI
Browse files Browse the repository at this point in the history
This didn't sed to be a problem because of a race condition in Fontship
that resulted in the rule being discarded anyway. When we fixed that bug
(using double expansion and a POSTFONTSHIPEVAL) we introduced
a dependency since we don't have XeLaTeX available in CI. This makes it
possible to regenerate locally but not required to generate packages.
  • Loading branch information
alerque committed Sep 19, 2024
1 parent 758651b commit eec5947
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion fontship.mk
Original file line number Diff line number Diff line change
Expand Up @@ -30,7 +30,7 @@ endef

define POSTFONTSHIPEVAL =

$$(DOCSDIR)/preview.pdf: $$(DOCSDIR)/preview.tex $$(STATICOTFS) | $$(BUILDDIR)
$$(DOCSDIR)/preview.pdf: $$(DOCSDIR)/preview.tex | $$(STATICOTFS) $$(BUILDDIR)
xelatex --interaction=batchmode -output-directory=$$(BUILDDIR) $$<
cp $(BUILDDIR)/$$(@F) $$@

Expand Down

0 comments on commit eec5947

Please sign in to comment.