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

Unable to build on Mac OS X El Capitan #60

Closed
guoway opened this issue Jul 15, 2016 · 3 comments
Closed

Unable to build on Mac OS X El Capitan #60

guoway opened this issue Jul 15, 2016 · 3 comments

Comments

@guoway
Copy link

guoway commented Jul 15, 2016

Environment:
OS : Mac OS X El Capitan
Node.js version : v6.3.0
pcsclite version : 0.4.9

Situation :
Following is the output when I was installing pcsclite.
kens-MacBook-Pro:node-pcsclite-master Ken$ npm install
npm WARN package.json pcsclite@0.4.9 license should be a valid SPDX license expression
> pcsclite@0.4.9 install /Users/ken/Documents/workspace/workspace_nodejs/node-pcsclite-master
> node-gyp rebuild

CXX(target) Release/obj.target/pcsclite/src/addon.o
CXX(target) Release/obj.target/pcsclite/src/pcsclite.o
../src/pcsclite.cpp:286:20: warning: comparison of integers of different signs: 'LONG' (aka 'int') and 'unsigned int' [-Wsign-compare]
if (result == SCARD_E_INSUFFICIENT_BUFFER) {
~~~~~~ ^ ~~~~~~~~~~~~~~~~~~~~~~~~~~~
1 warning generated.
CXX(target) Release/obj.target/pcsclite/src/cardreader.o
SOLINK_MODULE(target) Release/pcsclite.node

Can you help to solve it ?
Thank you ~

@guoway guoway changed the title Unable build on Mac OS X El Capitan Unable to build on Mac OS X El Capitan Jul 15, 2016
@santigimeno
Copy link
Owner

@guoway are you sure it's failing, from the logs it seems like a warning and not an error. (of course the warning should be fixed)

@guoway
Copy link
Author

guoway commented Jul 18, 2016

Sorry about that, I'm new to node.js, so cannot figure it clearly, now I know that's just a warning but error, sorry to bother you ~

@guoway guoway closed this as completed Jul 18, 2016
@santigimeno
Copy link
Owner

No problem. Thanks for the report though, I'll try to fix it asap

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