-
Notifications
You must be signed in to change notification settings - Fork 162
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
Subcommand watch fails when path to tectonic contains spaces #1003
Comments
I am unable to reproduce this on linux amd64. Might be localized to windows? |
I can no longer reproduce this under |
I was too fast in closing: Checking the code, the former is run directly while the latter is wrapped in a shell. |
Able to replicate this case. I see the error should be a simple fix where I should be able to append the quotes around the tectonic location and should work. I would be able to push a fix right now. |
This should be fixed in 0.14.1, which I released the other day. |
Unfortunately, the issue persists with 0.14.1: $ tectonic --version
Tectonic 0.14.1
$ tectonic -X watch -x "build --open"
note: "version 2" Tectonic command-line interface activated
[Running `"C:\Users\Markus Haug\scoop\apps\tectonic\current\tectonic.exe" -X build --open`]
Der Befehl "\"C:\Users\Markus Haug\scoop\apps\tectonic\current\tectonic.exe\"" ist entweder falsch geschrieben oder
konnte nicht gefunden werden.
[Finished Running. Exit Status: 1] Please excuse the German localization, I'm unable to change the locale for CMD. The error message roughly translates to “Command not found”. I've dug a bit deeper, and it appears that the problem lies with >uonsdu
Der Befehl "uonsdu" ist entweder falsch geschrieben oder
konnte nicht gefunden werden.
C:\Windows\System32>"uonsdu"
Der Befehl ""uonsdu"" ist entweder falsch geschrieben oder
konnte nicht gefunden werden.
C:\Windows\System32>\"uonsdu"
Der Befehl "\"uonsdu"" ist entweder falsch geschrieben oder
konnte nicht gefunden werden.
C:\Windows\System32>\"uonsdu\"
Der Befehl "\"uonsdu\"" ist entweder falsch geschrieben oder
konnte nicht gefunden werden. Apparently, the program name gets escaped somewhere down the stack. I will check the |
Apparently, the issue is |
I see that you're using windows and my tests were conducted on a Linux system that seemed to work. I'm sorry I couldn't really test on a windows machine cuz I didn't really have access to one. Just as a sanity check to make sure I'm doing this right, could you try running this on a linux machine and making sure everything works as intended? Maybe could help narrow this issue down to a windows specific one. |
I don't have access to a Linux machine at the moment, but testing version 0.14.1 on WSL worked fine. That makes me pretty certain that the issue is specific to CMD/Windows. There is also not a lot that tectonic can do about this, as the issue comes down to CMD parsing /C in a special way. This needs to be fixed in |
It appears that the path to tectonic is not properly quoted before it is passed to the shell for execution in the watch command. See an example below. Note that I translated the actual error message because I was unable to change the UI language for the error message.
The text was updated successfully, but these errors were encountered: