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

Tweak to README for Windows users #8

Merged
merged 1 commit into from
Feb 25, 2015

Conversation

jeffcharles
Copy link
Contributor

Ran into an issue on Windows where running parallelshell 'echo 1' resulted in:

''echo' is not recognized as an internal or external command,
operable program or batch file.
`'echo` failed with exit code 1

Running parallelshell "echo 1" worked. I'm pretty sure it's just due to how process.argv gets populated by Node. It seems to treat single quotes as part of the argument string.

@keithamus
Copy link
Collaborator

Aah the ol' double quotes. A lesson learned to test readme examples. Thanks @jeffcharles 👍

keithamus added a commit that referenced this pull request Feb 25, 2015
Tweak to README for Windows users
@keithamus keithamus merged commit b295fcb into darkguy2008:master Feb 25, 2015
estk pushed a commit to estk/parallelshell that referenced this pull request Sep 21, 2015
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.

2 participants