-
Notifications
You must be signed in to change notification settings - Fork 129
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
Build fails on Windows 7 #64
Comments
Thanks for report, let me take a look. |
Sorry, I have not virtual studio installed, os, it is quite difficult for me to testing that. Can you install other node c++ addon ? such as ws? |
Yes, ws and other C++ modules install fine for me. I have I know it's difficult if you don't have the environment setup, so let me know if there's anything I can try to get you more information. |
Can you clone the repo and do |
I think the closest thing that makes sense in Windows would be
Which gives this output:
|
You need install npm module |
This because the libgeoip target name is also "geoip". Then node-gyp is creating a solution file (binding.sln) containing two projetcs with same name. D:\Node-new\GeoIP>npm install
D:\Node-new\GeoIP>node "C:\Program Files (x86)\nodejs\node_modules\npm\bin\node-gyp-bin....\node_modules\node-gyp\bin\node-gyp.js" rebuild npm ERR! geoip@0.4.13-dev install: D:\Node-new\GeoIP> |
Despites the project name bug, Microsoft Visual Studio doesn't supports C99 extension (they are still developing this in MVS2013). |
why c99 ?, this module is mainly wrote in c++, it's just call libgeoip's functions. |
Although the module is in C++ it its using some features from C99, for instance variable-length arrays: |
Thanks for your clarifications. |
Just figured I'd add a tip: http://www.appveyor.com/ offers free windows-based CI testing, like Travis does for real operating systems. |
I would really like to see this on Windows |
Here's the output from npm:
The text was updated successfully, but these errors were encountered: