-
Notifications
You must be signed in to change notification settings - Fork 725
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
Better cross platform dev #109
Conversation
…you don't have to execute `cake build` any more.
Thanks for the work! Some of the commits are helpful. However, some changes make it hard for users to use this project. Please take into consideration that MOST of the users don't know anything about node and npm. They just want to download and run. And they wrote blogs to tell people how they thought this project should be used. Please check these blogs and make sure they will work as before after the pull request is merged. http://sae.chenshaoju.com/?p=14 |
@clowwindy What you mean by saying?
The changes I made only require people to have installed You merge my code, and run |
Bo, they don't even know how to use command line. All they know is to open local.js file with node.exe. |
@clowwindy Then, you should use the Github release page to upload the builded code for stable version: https://github.com/clowwindy/shadowsocks-nodejs/releases |
The source code is for the people who want to make a contribution. I could write a Cake task to help you create release zip file, and add a direct download link in the Readme. If you could give me the write permission to this project. |
Putting a big download button in the README seems a good idea. However, I can't ask people to rewrite blog articles like this one. When people search "how to use shadowsocks-nodejs", these blogs pop out in the first place. If the project has changed, people who have followed the blogs will fail. Anyway, thanks for the effort. What I need right now is something else rather than cleaning the code. Since node.js v0.10, shadowsock-nodejs is almost unusable for its huge memory footprint, due to the new buffer model. So I simply dropped out shadowsocks-nodejs development and focus on other versions. On a production environment with 2K concurrent connections, Python version only takes 20MB RAM (400MB on node.js) and uses half as CPU as node.js. So I've lost interest to work on this node.js version. Until this problem is solved, there will be no new version released. |
Well, I'll take time to dig into the memory issue, see if I can help. |
Thanks! That'll be appreciated. |
Checkout the Windows test: https://ci.appveyor.com/project/ysmood/shadowsocks-nodejs