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

Always add a slash after directory names in mkdir #10

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

zauguin
Copy link
Member

@zauguin zauguin commented Feb 14, 2024

Needed to properly create texmf-var after recent LuaTeX changes.

Needed to properly create texmf-var after recent LuaTeX changes.
@u-fischer
Copy link
Member

Well I'm not sure how to test if that is needed or not but trust that you know what you are doing ;-) But if this diverges from upstream the buildinfo.txt file probably needs to be updated.

@zauguin
Copy link
Member Author

zauguin commented Feb 17, 2024

Well I'm not sure how to test if that is needed or not but trust that you know what you are doing

As a bit of background: In new LuaTeX versions we are only allowed to run mkdir on (subdirs of the current dir and) paths starting with $TEXMFVAR followed by a slash. This works for creating luatex-cache inside of $TEXMFVAR, but when creating $TEXMFVAR itself it fails since $TEXMFVAR is not followed by anything then, especially not by a slash. Therefore $TEXMFVAR does not get created if it doesn't exist yet.

This is avoided with this change by just adding a slash at the end which is mostly ignored by mkdir, but makes the check happy.

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.

2 participants