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

Set TEXINPUTS in a system independent way #27

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

FG-TUM
Copy link

@FG-TUM FG-TUM commented Jul 22, 2023

Description

Problem

Prepending environment variables to commands works differently (if at all) on Windows. Sadly, you can't just do FOO=value myCommand. For this reason, the pdflatex alias command in latexmkrc does not work on Windows.

Fix

Use latexmk's ensure_path() function to be compatible with any operating system. See the official documentation (page 54).

Additional context

  • We don't need to append ':' to the new value, since ensure_path() prepends this value to the existing one and fills in the system-appropriate path separators.
  • Yes, I know there is a docker container, but if the tools we use are already system independent, why not employ them that way? :)

Tests

  • executed make pdf-local on Windows successfully
  • executed make pdf-local on Linux successfully

Use latexmk's 'ensure_path' function to be compatible with any operating system
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.

1 participant