Skip to content
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

npm install fails with with Node v12.5.0 #60

Closed
MarcusCalidus opened this issue Jul 2, 2019 · 1 comment
Closed

npm install fails with with Node v12.5.0 #60

MarcusCalidus opened this issue Jul 2, 2019 · 1 comment

Comments

@MarcusCalidus
Copy link

Hi,

upon installing [email protected] via npm i on Darwin 18.6.0 I receive following error:

../src/node_snap7_client.cpp:710:53: error: too few arguments to function call, single argument 'context' was not specified
    Nan::NewInstance(constructorHandle->GetFunction()).ToLocalChecked());
                     ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~ ^
/Users/marcowarm/.node-gyp/12.5.0/include/node/v8.h:5948:3: note: 'GetFunction' declared here
  V8_WARN_UNUSED_RESULT MaybeLocal<Function> GetFunction(
  ^
/Users/marcowarm/.node-gyp/12.5.0/include/node/v8config.h:351:31: note: expanded from macro 'V8_WARN_UNUSED_RESULT'
#define V8_WARN_UNUSED_RESULT __attribute__((warn_unused_result))
                              ^
2 errors generated.
make: *** [Release/obj.target/node_snap7/src/node_snap7_client.o] Error 1

@mathiask88
Copy link
Owner

fixed in 1.0.3

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants