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

Weird Electron Webpack Error #12

Closed
nmccready opened this issue Sep 13, 2018 · 2 comments
Closed

Weird Electron Webpack Error #12

nmccready opened this issue Sep 13, 2018 · 2 comments

Comments

@nmccready
Copy link

[0]   WARNING in ./node_modules/shell-env/node_modules/cross-spawn/index.js
[0]   Module not found: Error: Can't resolve 'spawn-sync' in '/Users/nem/code/sme/spear/sal/node_modules/shell-env/node_modules/cross-spawn'
[0]    @ ./node_modules/shell-env/node_modules/cross-spawn/index.js 32:26-47
[0]    @ ./node_modules/shell-env/node_modules/execa/index.js
[0]    @ ./node_modules/shell-env/index.js
[0]    @ ./node_modules/shell-path/index.js
[0]    @ ./node_modules/fix-path/index.js
[0]    @ ./src/main/index.js
[0]    @ multi ./node_modules/electron-webpack/out/electron-main-hmr/main-hmr ./src/main/index.js

Yet I do not see cross-spawn being used anywhere in this code base up from 0.3.0.

0.3.0 is where I am seeing this issue and cross-spawn does exist in our project dependencies but as a devDependency only. cross-spawn is also only being used by a unit test.

Anyway I was wondering if anyone had any ideas?

@nmccready
Copy link
Author

Ahh here is why. This was not fixed until 6.X .

@nmccready
Copy link
Author

So to resolve this y'all need to update execa in here to something newer using cross-spawn 6.X .

https://github.com/sindresorhus/execa/blob/master/package.json

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

1 participant