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

Set spawn opt shell as true to avoid ENOENT #1170

Merged
merged 1 commit into from
Dec 14, 2017

Conversation

sudo-suhas
Copy link
Contributor

This adds the option shell: true to the spawn invocation. We had identified and fixed this in #1133 (comment) but the change might have been lost during a merge/rebase. The change is not present in the squash-merge commit ecd68c4. This change should fix it for everyone.

Related issue - vuejs/vue-cli#681.
Closes #1168.

@LinusBorg
Copy link
Contributor

Ah, damn - totally didn't notice. Seems I somehow lost this during a merge/trebase indeed.

@LinusBorg LinusBorg merged commit 4f564a2 into vuejs-templates:develop Dec 14, 2017
@sudo-suhas sudo-suhas deleted the spawn-shell-opt branch December 14, 2017 12:11
LinusBorg added a commit that referenced this pull request Dec 14, 2017
* develop:
  correct wording for #1155
  bump version to 1.2.7
  remove eventsource-polyfill (#1169)
  Remove useless code when lint is not chosen (#1165)
  Set spawn opt shell as true to avoid ENOENT (#1170)
  restore gitignore
  [WIP] Accurate CompleteMsg when not using autoInstall (fix #1157) (#1158)
  close #1155
shenron pushed a commit to shenron/webpack that referenced this pull request Mar 20, 2018
shenron pushed a commit to shenron/webpack that referenced this pull request Mar 20, 2018
* develop:
  correct wording for vuejs-templates#1155
  bump version to 1.2.7
  remove eventsource-polyfill (vuejs-templates#1169)
  Remove useless code when lint is not chosen (vuejs-templates#1165)
  Set spawn opt shell as true to avoid ENOENT (vuejs-templates#1170)
  restore gitignore
  [WIP] Accurate CompleteMsg when not using autoInstall (fix vuejs-templates#1157) (vuejs-templates#1158)
  close vuejs-templates#1155
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.

Auto-Install fails on some OS/node combinations
2 participants