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

Main: Run executables from the temporary directory. #87

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

Conversation

MaxKoll
Copy link

@MaxKoll MaxKoll commented May 19, 2021

As of May 2020, MiKTeX does no longer support short filenames (aka 8.3 filenames), so if the path to the temporary directory contains short filenames, LaTeX compilation will fail. Unfortunately, in the current framework it seems to be impossible to obtain the full path with only long filenames easily.

Resolve this by launching a shell that changes into the temporary directory and then calls the executable from there.

Tested on:

  • Windows 10 Pro (20H2), Thunderbird 78.10.1, MiKTeX 21.2
  • Xubuntu 21.04, Thunderbird 78.8.1, TeX Live 2020

See #86 for a bit more elaboration on the problem.

@sphh sphh requested a review from protz May 19, 2021 17:03
@MaxKoll MaxKoll force-pushed the pr-exec-from-temp branch from f76efb2 to 202f771 Compare May 23, 2021 20:55
@MaxKoll
Copy link
Author

MaxKoll commented May 23, 2021

Fixed a mistake (shell_process.exitValue -> exitValue) and some formatting details.

As of May 2020, MiKTeX does no longer support short filenames (aka 8.3 filenames), so if the path to the temporary directory contains short filenames, LaTeX compilation will fail. Unfortunately, in the current framework it seems to be impossible to obtain the full path (with only long filenames) easily.

Resolve this by launching a shell that changes into the temporary directory and then calls the executable from there.
@MaxKoll MaxKoll force-pushed the pr-exec-from-temp branch from 202f771 to adf0caa Compare May 29, 2021 02:17
@MaxKoll
Copy link
Author

MaxKoll commented May 29, 2021

Some restructuring and reformatting to improve clarity. Program flow did not change.

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