Unable to compile for electron target: caused by recent node-addon-api 3.1.0 => 3.2.0 update #386

vladimiry · 2021-05-19T15:30:21Z

Steps to Reproduce

HOME=~/.electron-gyp node-gyp rebuild --target=13.0.0-beta.27 --arch=x64 --dist-url=https://electronjs.org/headers

Gives:

  CXX(target) Release/obj.target/keytar/src/async.o
In file included from ../node_modules/node-addon-api/napi.h:2725,
                 from ../src/async.cc:4:
../node_modules/node-addon-api/napi-inl.h: In member function ‘bool Napi::Object::Freeze()’:
../node_modules/node-addon-api/napi-inl.h:1393:24: error: ‘napi_object_freeze’ was not declared in this scope; did you mean ‘napi_object_expected’?
 1393 |   napi_status status = napi_object_freeze(_env, _value);
      |                        ^~~~~~~~~~~~~~~~~~
      |                        napi_object_expected
../node_modules/node-addon-api/napi-inl.h: In member function ‘bool Napi::Object::Seal()’:
../node_modules/node-addon-api/napi-inl.h:1399:24: error: ‘napi_object_seal’ was not declared in this scope; did you mean ‘napi_object’?
 1399 |   napi_status status = napi_object_seal(_env, _value);
      |                        ^~~~~~~~~~~~~~~~
      |                        napi_object
make: *** [keytar.target.mk:121: Release/obj.target/keytar/src/async.o] Error 1

Works fine if I downgrade node-addon-api 3.2.0 => 3.1.0.

The text was updated successfully, but these errors were encountered:

shiftkey · 2021-05-19T16:15:03Z

Not sure I understand the issue, as we're using 3.1.0 in the lockfile:

node-keytar/package-lock.json

Lines 2206 to 2210 in 1248408

    
           "node-addon-api": { 
        
             "version": "3.1.0", 
        
             "resolved": "https://registry.npmjs.org/node-addon-api/-/node-addon-api-3.1.0.tgz", 
        
             "integrity": "sha512-flmrDNB06LIl5lywUz7YlNGZH/5p0M7W28k8hzd9Lshtdh1wshD2Y+U4h9LD6KObOy1f+fEVdgprPrEymjM5uw==" 
        
           },

And our dependency is not 3.2.0 specifically:

node-keytar/package.json

Line 53 in 1248408

"node-addon-api": "^3.0.0",

There's some extra context here about how we're prebuild-ing for N-API now #331 so I'm not sure if that's somehow affected things

shiftkey · 2021-05-19T16:18:16Z

I was also able to do this successfully with the latest version on macOS (using our version of node-gyp, not the global one):

$ ./node_modules/.bin/node-gyp rebuild --target=13.0.0-beta.27 --arch=x64 --dist-url=https://electronjs.org/headers
gyp info it worked if it ends with ok
gyp info using node-gyp@7.1.2
gyp info using node@14.15.1 | darwin | x64
gyp info find Python using Python version 3.9.5 found at "/usr/local/opt/python@3.9/bin/python3.9"
gyp http GET https://electronjs.org/headers/v13.0.0-beta.27/node-v13.0.0-beta.27.tar.gz
gyp http 200 https://electronjs.org/headers/v13.0.0-beta.27/node-v13.0.0-beta.27.tar.gz
gyp http GET https://electronjs.org/headers/v13.0.0-beta.27/SHASUMS256.txt
gyp http 200 https://electronjs.org/headers/v13.0.0-beta.27/SHASUMS256.txt
gyp info spawn /usr/local/opt/python@3.9/bin/python3.9
gyp info spawn args [
gyp info spawn args   '/Users/shiftkey/src/node-keytar/node_modules/node-gyp/gyp/gyp_main.py',
gyp info spawn args   'binding.gyp',
gyp info spawn args   '-f',
gyp info spawn args   'make',
gyp info spawn args   '-I',
gyp info spawn args   '/Users/shiftkey/src/node-keytar/build/config.gypi',
gyp info spawn args   '-I',
gyp info spawn args   '/Users/shiftkey/src/node-keytar/node_modules/node-gyp/addon.gypi',
gyp info spawn args   '-I',
gyp info spawn args   '/Users/shiftkey/Library/Caches/node-gyp/13.0.0-beta.27/include/node/common.gypi',
gyp info spawn args   '-Dlibrary=shared_library',
gyp info spawn args   '-Dvisibility=default',
gyp info spawn args   '-Dnode_root_dir=/Users/shiftkey/Library/Caches/node-gyp/13.0.0-beta.27',
gyp info spawn args   '-Dnode_gyp_dir=/Users/shiftkey/src/node-keytar/node_modules/node-gyp',
gyp info spawn args   '-Dnode_lib_file=/Users/shiftkey/Library/Caches/node-gyp/13.0.0-beta.27/<(target_arch)/node.lib',
gyp info spawn args   '-Dmodule_root_dir=/Users/shiftkey/src/node-keytar',
gyp info spawn args   '-Dnode_engine=v8',
gyp info spawn args   '--depth=.',
gyp info spawn args   '--no-parallel',
gyp info spawn args   '--generator-output',
gyp info spawn args   'build',
gyp info spawn args   '-Goutput_dir=.'
gyp info spawn args ]
gyp info spawn make
gyp info spawn args [ 'BUILDTYPE=Release', '-C', 'build' ]
  CXX(target) Release/obj.target/keytar/src/async.o
  CXX(target) Release/obj.target/keytar/src/main.o
  CXX(target) Release/obj.target/keytar/src/keytar_mac.o
  SOLINK_MODULE(target) Release/keytar.node
gyp info ok

vladimiry · 2021-05-19T16:30:28Z

Not sure I understand the issue, as we're using 3.1.0 in the lockfile:

Lock file doesn't make sense if the module is installed from npm (since lock files don't get published to npm), so the new node-addon-api version got into the game by "node-addon-api": "^3.0.0" range. I just don't use prebuilds and this is how the issue got noticed.

I was also able to do this successfully with the latest version on macOS (using our version of node-gyp, not the global one):

I've tried both 8.0.0 (globally installed) and ./node_modules/.bin/node-gyp (7.1.2), the result is the same. Try it with node-addon-api@3.2.0.

latest version on macOS

Maybe the issue is OS-specific. I don't have macOS system to test it in my own.

PalmerAL · 2021-06-11T20:06:20Z

I'm having the same problem; I think it only fails with newer versions of node. With node 14.11.0 installed, I can build successfully; with later versions, it fails.

~~14.12.0~~ 15.12.0 is the first version to support NAPI v8 (source), and the node-addon-api commit that introduced the error is conditioned on v8 being available: nodejs/node-addon-api@77350ee. However, I still don't really understand why it fails with NAPI v8.

@shiftkey - if you install the latest version of node and then try to build, can you see the issue?

atom/node-keytar#386 (comment)

gaetandezeiraud · 2021-06-25T10:33:37Z

I'm on Node v14.17.1 and I have the same problem @PalmerAL

vladimiry · 2021-07-01T17:46:28Z

I just pinned keytar/node-addon-api@3.1.0 as a workaround for use with node 15.

vladimiry · 2021-07-02T08:55:42Z

Btw, node-addon-api v4 was released about 2 weeks ago and it's also affected.

Wiltuk · 2021-07-02T19:27:34Z

I was having this issue on Windows with node 14.17.1 - I found the error does not occur with keytar version 7.6.0 so it seems to be something in the 7.7.0 release.

gaetandezeiraud · 2021-07-06T15:46:31Z

@Wiltuk I have the same problem with keytar 7.6.0 on my side. Event after deleting node_modules and lock file.

felipe-linares · 2021-07-12T20:48:20Z

+1 have this exact issue on macOS. Can fix by running on node 14 <= 14.16.1, but our team needs 15+ for ARM builds of our electron application. Any insight on this would be highly appreciated

EDIT: It seems that what @PalmerAL said is spot on. Using Node 15 <15.12.0 works fine, so it must be something with NAPIv8 is causing this.

gaetandezeiraud · 2021-07-14T08:36:39Z

No workaround with the lastest LTS version of node?

vladimiry · 2021-07-14T08:42:29Z

@Brouilles see the issue title, ie downgrading node-addon-api to 3.1.0 is a workaround.

felipe-linares · 2021-07-27T21:37:12Z

Any update on this? Some of our other dependencies (lodash in particular) don't support node 15, only node 14/16. But we need 15+ for ARM builds so 16 would be our only option, but because of node-addon-api issues it doesn't work on node 16...

Ranjith-Eswaran-G · 2021-12-16T07:18:29Z

@vladimiry , I am also facing the same issue. I updated the keytar version from 7.3.0 to 7.7.0. I also installed node-addon-api version 3.1.0. But still I am facing the same issue. May I know what was your work around?

vladimiry · 2021-12-16T07:41:31Z

May I know what was your work around?

The workaround was in downgrading node-addon-api to 3.1.0 but I removed downgrade in my project some time ago as the issue got fixed somewhere upstream (currently used node-addon-api is 3.2.1).

So I believe the issue can be closed now.

Ranjith-Eswaran-G · 2021-12-16T07:46:41Z

The workaround was in downgrading node-addon-api to 3.1.0 but I removed downgrade in my project some time ago as the issue got fixed somewhere upstream

May I know your current Electron and Node version?

vladimiry · 2021-12-16T07:49:56Z

Node 16.13.1 and @electron 15.3.2 & 17.0.0-alpha.4 (was fine on @electron v16 too).

Ranjith-Eswaran-G · 2021-12-16T07:53:26Z

Thanks for your response. But I am getting the error you mentioned at the top even after downgrading node-addon-api to 3.1.0. It works fine only when I use the node version 14.11.0.

vladimiry · 2021-12-16T08:07:50Z

I guess node-gyp version also might be a part of the stack, at my side it's node-gyp@8.2.0.

PalmerAL mentioned this issue Jun 6, 2021

Keytar module doesn't build successfully minbrowser/min#1636

Open

PalmerAL added a commit to minbrowser/min that referenced this issue Jun 11, 2021

downgrade node to fix build issue

39fc854

atom/node-keytar#386 (comment)

gaetandezeiraud mentioned this issue Jun 25, 2021

Can't build keytar with Electron Windows #399

Closed

PalmerAL mentioned this issue Oct 7, 2021

Figure out a migration path for the native password manager minbrowser/min#1761

Closed

PalmerAL mentioned this issue Jan 7, 2022

Issues building on M1 (potentially NOOB) minbrowser/min#1850

Closed

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Unable to compile for electron target: caused by recent node-addon-api 3.1.0 => 3.2.0 update #386

Unable to compile for electron target: caused by recent node-addon-api 3.1.0 => 3.2.0 update #386

vladimiry commented May 19, 2021

shiftkey commented May 19, 2021

shiftkey commented May 19, 2021 •

edited

Loading

vladimiry commented May 19, 2021 •

edited

Loading

PalmerAL commented Jun 11, 2021 •

edited

Loading

gaetandezeiraud commented Jun 25, 2021

vladimiry commented Jul 1, 2021 •

edited

Loading

vladimiry commented Jul 2, 2021

Wiltuk commented Jul 2, 2021 •

edited

Loading

gaetandezeiraud commented Jul 6, 2021

felipe-linares commented Jul 12, 2021 •

edited

Loading

gaetandezeiraud commented Jul 14, 2021

vladimiry commented Jul 14, 2021

felipe-linares commented Jul 27, 2021

Ranjith-Eswaran-G commented Dec 16, 2021

vladimiry commented Dec 16, 2021 •

edited

Loading

Ranjith-Eswaran-G commented Dec 16, 2021

vladimiry commented Dec 16, 2021 •

edited

Loading

Ranjith-Eswaran-G commented Dec 16, 2021

vladimiry commented Dec 16, 2021

Unable to compile for electron target: caused by recent node-addon-api 3.1.0 => 3.2.0 update #386

Unable to compile for electron target: caused by recent node-addon-api 3.1.0 => 3.2.0 update #386

Comments

vladimiry commented May 19, 2021

Steps to Reproduce

shiftkey commented May 19, 2021

shiftkey commented May 19, 2021 • edited Loading

vladimiry commented May 19, 2021 • edited Loading

PalmerAL commented Jun 11, 2021 • edited Loading

gaetandezeiraud commented Jun 25, 2021

vladimiry commented Jul 1, 2021 • edited Loading

vladimiry commented Jul 2, 2021

Wiltuk commented Jul 2, 2021 • edited Loading

gaetandezeiraud commented Jul 6, 2021

felipe-linares commented Jul 12, 2021 • edited Loading

gaetandezeiraud commented Jul 14, 2021

vladimiry commented Jul 14, 2021

felipe-linares commented Jul 27, 2021

Ranjith-Eswaran-G commented Dec 16, 2021

vladimiry commented Dec 16, 2021 • edited Loading

Ranjith-Eswaran-G commented Dec 16, 2021

vladimiry commented Dec 16, 2021 • edited Loading

Ranjith-Eswaran-G commented Dec 16, 2021

vladimiry commented Dec 16, 2021

shiftkey commented May 19, 2021 •

edited

Loading

vladimiry commented May 19, 2021 •

edited

Loading

PalmerAL commented Jun 11, 2021 •

edited

Loading

vladimiry commented Jul 1, 2021 •

edited

Loading

Wiltuk commented Jul 2, 2021 •

edited

Loading

felipe-linares commented Jul 12, 2021 •

edited

Loading

vladimiry commented Dec 16, 2021 •

edited

Loading

vladimiry commented Dec 16, 2021 •

edited

Loading