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

Make Latexrun work on Windows #50

Open
wants to merge 3 commits into
base: master
Choose a base branch
from

Conversation

onnodb
Copy link

@onnodb onnodb commented Apr 18, 2017

These changes enabled me to use latexrun on Windows (with MikTeX). Just some minor fixes of path and stdout handling. At this point I can't guarantee that everything works as intended, but it seems to do the trick for now. I hope this helps someone out there!

In any case, thanks for putting latexrun out here. It's a very helpful utility :-)

onnodb added 3 commits April 18, 2017 22:00
Convert Windows line endings into Unix ones, for correct processing downstream.
`os.rename` doesn't behave correctly on Windows, and causes a `WinError` exception. `os.replace` seems to be the better way to do atomic renames, and also works on Windows.
Use `os.path.isabs` instead of manually detecting an initial forward slash. This also works on Windows.
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