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

folder path bug #81

Open
Tareku99 opened this issue Mar 8, 2021 · 6 comments
Open

folder path bug #81

Tareku99 opened this issue Mar 8, 2021 · 6 comments
Assignees

Comments

@Tareku99
Copy link

Tareku99 commented Mar 8, 2021

i put the folder on my desktop and run the line in windows.
I got an error saying "path not excepted at this time". I decided to move the folder with the .bat to a root directory and it worked fine. I encounter this issue a lot, my username has a space in between; example "steve bob". I assume the .bat file does not account for random spaces like that in a path when executing.

@gimsieke gimsieke self-assigned this Mar 8, 2021
@gimsieke
Copy link
Contributor

gimsieke commented Mar 8, 2021

This observation is correct, @Tareku99.

I’m going to try to add support for spaces in paths in d2t.bat and in calabash/calabash.bat.

(This issue is a duplicate of #66, and it is related to #18 that is about d2t)

gimsieke added a commit to transpect/calabash-frontend that referenced this issue Mar 8, 2021
gimsieke added a commit that referenced this issue Mar 8, 2021
@gimsieke
Copy link
Contributor

gimsieke commented Mar 8, 2021

Please check out the latest version (using git pull --recurse-submodules) and see whether space handling has improved (here’s hoping that it has).

@Tareku99
Copy link
Author

Please check out the latest version (using git pull --recurse-submodules) and see whether space handling has improved (here’s hoping that it has).

I pulled using your command and run a test random file in a random path that has a space. It is not working from what I can see.
LOG:
'"C:/ds ds/docx2tex//calabash/calabash.bat"' is not recognized as an internal or external command,
operable program or batch file.

@Tareku99
Copy link
Author

Please check out the latest version (using git pull --recurse-submodules) and see whether space handling has improved (here’s hoping that it has).

I pulled using your command and run a test random file in a random path that has a space. It is not working from what I can see.
LOG:
'"C:/ds ds/docx2tex//calabash/calabash.bat"' is not recognized as an internal or external command,
operable program or batch file.

I clone again using the -recursive flag (which I did not use before) and it works fine. Was it my error from the start ?

@gimsieke
Copy link
Contributor

I don’t know. Git and Windows batch files are mysteries to me, too.

@gimsieke
Copy link
Contributor

Maybe this finding is relevant here: https://github.com/transpect/docx2tex/blob/master/d2t.bat#L8

It seems that if you invoke batch files using forward slashes, and if the directory that the batch file is in contains spaces, the command is not recognized. Then you need to prepend cmd /c to the batch file name.

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

No branches or pull requests

2 participants