-
-
Notifications
You must be signed in to change notification settings - Fork 1.4k
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
webpack-dev-server will not work on Windows due to its dependency on node-gyp #229
Comments
Anyone here have any idea why node-gyp is essential to webpack-dev-server? |
This is necessary, because otherwise the dev-server can't livereload itself. |
I think the native stuff in bufferutils is an optional dependency and you should be able to install it without it. Otherwise send an issue to the engine.io repo. Here is a guide how to make node-gyp work on windows. As far as I remember it was straight-forward: https://github.com/TooTallNate/node-gyp/#installation |
Hey @SpaceK33z ' hi @sokra |
HI do you found a solution for this? best regards |
no solutions. and as you're saying - definitely not "straight forward"... |
in my daily job i use Windows and i agree that node-gyp is a nigthmare on windows, but after follow https://github.com/TooTallNate/node-gyp/#installation i have everything working. also this issue should go in the engine.io repo.. it's not webpack's fault |
I'm glad for you node-gyp works on your system. It doesn't on mine and on others. |
I use windows, and it works fine. I have two machines one with VS and Python so the node-gyp works and one without both of them and webpack-dev-server still works fine. The native dependencies are optional and everything works also without them. If you doesn't work for you that propably an issue of an incorrect node-gyp configuration and not a missing one.
Feel free to send a PR. The public interface doesn't expose socket.io. |
I don't use Windows, but unfortunately other team members do so I also feel the pain. We used to run into the node-gyp nightmare every time someone new tries to run their first maven build. team member: "Hey, the maven build is failing - its complaining something about npm. I have npm installed, what's up?" I now leave all node-gyp dependent packages out of our package.json dev dependency list. |
I've started trying to convert webpack-dev-server to use sockjs. Will create a PR when it's done. I often get complaints from other co-workers as well, because node-gyp requires you to install some build packages on Debian. This is easily fixable, but I think a project should work immediately after a |
This removes the node-gyp dependency :). Fixes webpack#229, webpack#276, fixes webpack#195, fixes webpack#267, fixes webpack#258 and fixes webpack#242
Hi I tested it on my environment and it works fine. Thanks a lot. |
Replace socket.io with SockJS as discussed in #229
Awesome work @SpaceK33z!!! |
Hi Ajar-Ajar, SpaceK33z, |
I'm a C++ developer, and I agree with Ajar-AJar that it makes no sense. |
The only dependency that requires node gyp is watchpack now (indirectly, via chokidar). There is an open PR in watchpack that removes that switches chokidar to sane, removing the node gyp dep (am on mobile now, so no link). Please help test that PR there instead of complaining here. |
Hi, I need to ask if webpack-dev-server's dependency on node-gyp is absolutely necessary?
node-gyp is a nightmare for many node developers that develop on a windows machine.
it needs a C++ compiler which forces you to download visual studio and all sorts of service packs that got nothing to do with node development. in our case - after a 1.6 Gigs of installations and dependencies we were still unable to make node-gyp work on windows...
Why would webpack-dev-server however need node-gyp as a dependency?
As far as I know it has an express http server with socket.io... the webpack package alone doesn't require node-gyp, so why the webpack-dev-server need it?
Can you let it go? it creates a lot of unnecessary pain for developers...
Cheers
Ajar
The text was updated successfully, but these errors were encountered: