-
Notifications
You must be signed in to change notification settings - Fork 48
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
Win32 - fix netsh command #17
base: master
Are you sure you want to change the base?
Conversation
Hmm, well I'm not sure why this should be necessary. Currently, Have you verified the output of |
I didn't user autoFindInterface since I have more than 1 wireless adapters installed and I have to control many of them at a time. Here was my code that have a problem with the original code.
|
Ah, ok. Windows support is probably the weakest of the 3 OS in this package because I rarely use it myself. The issue here is you are supplying in this example something like You can see how those spaces can break the command. Assuming
|
Ok, I'm sorry -- I misread your changes re: I will test this on a Win32 machine when I can!
|
the command of netsh wasn't
|
Yes, see above; I realized my mistake shortly before you replied! I understand now what you've accomplished -- but I need to verify it on the next win32 system I can find. |
These old netsh command doesn't work. It can work only if your network interface name was "wlan".
I don't know how to compile those file. But I change it in .coffee. Not in .js like in my work.