OpenZWave / node-openzwave-shared

OpenZWave addon for Node.js (all versions) including management and security functions
Other
199 stars 113 forks source link

Building master branch fails on Node v12.9.0 #338

Closed CliffS closed 5 years ago

CliffS commented 5 years ago

The relevant part of the log appears to be:

../src/openzwave.cc: In function ‘void OZW::init(v8::Local<v8::Object>, v8::Local<v8::Object>)’:
../src/openzwave.cc:184:55: error: ‘GetValueAsBitSet’ is not a member of ‘OZW::OZW’
   Nan::SetPrototypeMethod(t, "getValueAsBitSet", OZW::GetValueAsBitSet);
                                                       ^~~~~~~~~~~~~~~~
../src/openzwave.cc:185:49: error: ‘SetBitMask’ is not a member of ‘OZW::OZW’
   Nan::SetPrototypeMethod(t, "setBitMask", OZW::SetBitMask);
                                                 ^~~~~~~~~~
../src/openzwave.cc:186:49: error: ‘GetBitMask’ is not a member of ‘OZW::OZW’
   Nan::SetPrototypeMethod(t, "getBitMask", OZW::GetBitMask);
                                                 ^~~~~~~~~~
../src/openzwave.cc:187:52: error: ‘GetBitSetSize’ is not a member of ‘OZW::OZW’
   Nan::SetPrototypeMethod(t, "getBitSetSize", OZW::GetBitSetSize);
                                                    ^~~~~~~~~~~~~
In file included from /home/cliff/src/git/house/node_modules/nan/nan.h:54,
                 from ../src/openzwave.hpp:28,
                 from ../src/openzwave.cc:18:
../src/openzwave.cc: At global scope:
ekarak commented 5 years ago

You must either upgrade you OpenZWave library, or remove duplicate headers for OpenZWave. These calls were added with Openzwave 1.6 and you tried to link against an older version of the library that doesn't support these calls.