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

Fails to rebuild for electron 28+ (v8 12+) #101

Open
chrbongardt opened this issue Feb 28, 2024 · 4 comments
Open

Fails to rebuild for electron 28+ (v8 12+) #101

chrbongardt opened this issue Feb 28, 2024 · 4 comments

Comments

@chrbongardt
Copy link

V8 v12 brings some breaking changes to the ObjectTemplate::SetAccessor signature. NAN is working on a fix: nodejs/nan#966.

As soon as the fixing NAN version is released I will open a PR. Just wanted to have this tracked.

Thanks!

@chrbongardt
Copy link
Author

Done, I just created PR #102 that should fix this.

@chrbongardt
Copy link
Author

While this get's merged and released by @santigimeno you can use my branch by doing
npm i [email protected]:chrbongardt/node-pcsclite.git#master

  • Tested up to electron v29!

@rkeithhill-keysight
Copy link

Huh, nan 2.19.0 does not fix this issue for Electon 28 builds. I still get:

'v8::ObjectTemplate::SetAccessor': no overloaded function could convert all the argument types

@crudolf
Copy link

crudolf commented Oct 12, 2024

@santigimeno would be awesome to have new release here ❤️

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

3 participants