Skip to content

Fix path to ANT when spaces are present.#36

Merged
SlavaRa merged 1 commit intoSlavaRa:masterfrom
Neverbirth:master
Jul 20, 2016
Merged

Fix path to ANT when spaces are present.#36
SlavaRa merged 1 commit intoSlavaRa:masterfrom
Neverbirth:master

Conversation

@Neverbirth
Copy link
Copy Markdown
Contributor

Path to ANT was missing the quotes around it, and then you also need to put the whole command for cmd /c between quotes too.

NOTE: Spacing inconsistencies also fixed.

@SlavaRa
Copy link
Copy Markdown
Owner

SlavaRa commented Jul 20, 2016

Thanks :)

@SlavaRa SlavaRa merged commit 2207070 into SlavaRa:master Jul 20, 2016
@gene-pavlovsky
Copy link
Copy Markdown
Contributor

The file manager I use, FAR, uses cmd /s /c "command line" to execute user-entered command line. This preserves quotes in the command line itself (see cmd /? for devilish complicated logic of how it treats quotes on the command line). Maybe /s should be added? What about Linux support in the future?

@Neverbirth
Copy link
Copy Markdown
Contributor Author

Neverbirth commented Jul 22, 2016

For the /s while googling I saw a guy that supposedly had problems with it and had to resort to double quotes, but I don't know if it's fully trustworthy.

For Linux and Mac, you can leave the path empty and it will just call ant, I don't know how double quotes work there. But I think FD should have a global setting for something like the terminal executable to be used in the application.

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.

3 participants