-
Notifications
You must be signed in to change notification settings - Fork 504
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
nan_callbacks.h:55:23: error: ‘AccessorSignature’ is not a member of ‘v8’ #942
Comments
Same problem when trying to compile for Electron v20
|
same problem, using
|
A PR has already been sent to fix this - #941. |
Confirmed, this PR fix my problem when rebuild electron with node-pty. |
confirmed as well! thank! |
Let's keep this open for now. It will get closed automatically when the PR lands! |
In case someone is looking for workarounds until this is fixed, take a look to: electron/electron#35193 |
I am using a midi package. Recently I updated to node v 18.10.0 and nw v0.69.1. When I rebuild:
|
@RichardFoss I didn't follow that approach and it works in my case. This is what I did:
"scripts": {
"postinstall": "patch-package || true",
},
"devDependencies": {
"patch-package": "^6.4.7",
},
diff --git a/node_modules/nan/nan.h b/node_modules/nan/nan.h
index 9e7c59e..0326246 100644
--- a/node_modules/nan/nan.h
+++ b/node_modules/nan/nan.h
@@ -2516,8 +2516,7 @@ inline void SetAccessor(
, SetterCallback setter = 0
, v8::Local<v8::Value> data = v8::Local<v8::Value>()
, v8::AccessControl settings = v8::DEFAULT
- , v8::PropertyAttribute attribute = v8::None
- , imp::Sig signature = imp::Sig()) {
+ , v8::PropertyAttribute attribute = v8::None) {
HandleScope scope;
imp::NativeGetter getter_ =
@@ -2550,9 +2549,6 @@ inline void SetAccessor(
, obj
, settings
, attribute
-#if (NODE_MODULE_VERSION < NODE_18_0_MODULE_VERSION)
- , signature
-#endif
);
}
diff --git a/node_modules/nan/nan_callbacks.h b/node_modules/nan/nan_callbacks.h
index 53ede84..ea81e45 100644
--- a/node_modules/nan/nan_callbacks.h
+++ b/node_modules/nan/nan_callbacks.h
@@ -52,8 +52,6 @@ typedef void(*IndexQueryCallback)(
const PropertyCallbackInfo<v8::Integer>&);
namespace imp {
-typedef v8::Local<v8::AccessorSignature> Sig;
-
static const int kDataIndex = 0;
static const int kFunctionIndex = 1; |
Thanks for this comprehensive comment! I have gone down a different path now after feedback from the midi group. I am using an updated midi implementation - read here - https://github.com/julusian/node-midi#readme, and this is working well for me! The nan issue is not present with this implementation. |
The previous version had a bug that complained about 'AccessorSignature' not being part of v8, as reported here: nodejs/nan#942
This solved it for me
It just upgrades the libraries in package.json to the latest versions |
same here :/ |
how to add this patch and where? |
still happens when tried to move to pnpm... |
It works for me if I use yarn
|
fixes crash seen in nodejs/nan#942
fixes crash seen in nodejs/nan#942
If someone is wondering Node.JS 20 build failure use similar approach as above with updated version since some package did not update it on their latest like:
|
i am not sure i am posting this in the correct place, but i am facing the below issue and i am not sure how to proceed further on this.
appreciate any push in the right direction.
The text was updated successfully, but these errors were encountered: