-
Notifications
You must be signed in to change notification settings - Fork 123
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
W7 x64 MSBuild #102
Comments
Can you try updating the nan version in package.json, then run
The VS version is auto-detected by gyp / node-gyp (the build system for node.js add-ons), not node-iconv. You can override it by passing |
Can you explain howto load the package, edit the package, build the package, how to break |
Just clone the repo; the package.json in the usual place, the top-level directory. You run |
OK. I've updated package.json, for ´nan´ and ´tap´: C:\develop\nodejs>cd node-iconv C:\develop\nodejs\node-iconv>npm install
C:\develop\nodejs\node-iconv>node "C:\Program Files\nodejs\node_modules\npm\bin C:\develop\nodejs\node-iconv> |
@jmparatte I landed some changes in e7053b2 and 2fb992b that should reduce the number of warnings. I can't upgrade nan right now because of nodejs/nan#222. I'm going to tentatively close this as fixed pending a new release. |
Hi,
I'm configuring a new pc and reinstalling nodejs.
In the past I installed iconv with npm without problems, last time 2014-04-23.
Today it's a vera hard task...
First. I installed Visual Studio 2013 for Desktop, necessary to install another package: serialport. No problems, easy.
1st try to install iconv:
2nd try to install iconv:
As you can read it, there are a lot of warnings compiling nan.h file.
How to solve them ?
Can you update the package to use VS2013 instead of 2010 ?
Best regards.
The text was updated successfully, but these errors were encountered: