search

TooTallNate / node-time

"time.h" bindings for Node.js
MIT License
381 stars 84 forks source link

Build error: error C2065: 'tzname': undeclared identifier #83

Closed jianglinchun closed 8 years ago

jianglinchun commented 8 years ago

..\src\time.cc(46): error C2065: 'tzname': undeclared identifier [E:\code\github\Countly\countly\node_modules\time\build\time.vcxproj] ..\src\time.cc(46): error C2228: left of '.ToLocalChecked' must have class/struct/union [E:\code\github\Countly\countly\node_modules\time\build\time.vcxproj] ..\src\time.cc(46): error C2661: 'Nan::Set': no overloaded function takes 2 arguments [E:\code\github\Countly\countly\node_modules\time\build\time.vcxproj] I'm fully awared that windows is not supported. I had installed the time module successfully on linux. Will those errors be fixed??? ..\src\time.cc(52): error C2065: 'timezone': undeclared identifier [E:\code\github\Countly\countly\node_modules\time\build\time.vcxproj] ..\src\time.cc(52): error C2661: 'Nan::Set': no overloaded function takes 2 arguments [E:\code\github\Countly\countly\node_modules\time\build\time.vcxproj] ..\src\time.cc(56): error C2065: 'daylight': undeclared identifier [E:\code\github\Countly\countly\node_modules\time\build\time.vcxproj] ..\src\time.cc(56): error C2661: 'Nan::Set': no overloaded function takes 2 arguments [E:\code\github\Countly\countly\node_modules\time\build\time.vcxproj]

jianglinchun commented 8 years ago

I solved this problem by made those changes: https://github.com/jianglinchun/node-time/commit/aca7dc57c00d60cf320b7ffaf4dd951775af1017 I had tested under window10 and node v4.2.2.