discord / erlpack

High Performance Erlang Term Format Packer
MIT License
220 stars 65 forks source link

Update nan package to fix build issue #58

Open JMTK opened 1 year ago

JMTK commented 1 year ago

The postbuild script to rebuild those packages when installing discord.js (on Node 19 for me) with:

../../nan/nan_callbacks.h:55:23: error: ‘AccessorSignature’ is not a member of ‘v8’
   55 | typedef v8::Local<v8::AccessorSignature> Sig;
      |                       ^~~~~~~~~~~~~~~~~
../../nan/nan_callbacks.h:55:40: error: template argument 1 is invalid
   55 | typedef v8::Local<v8::AccessorSignature> Sig;
      | 

Changelog: https://github.com/nodejs/nan/compare/v2.16.0...v2.17.0

yarn why nan showed that those packages prefer the older versions, simply adding to my package.json

"resolutions": {
    "nan": "2.17.0"
},

fixes the issue. Not sure if there's anything better I could do

Resolves #57

JMTK commented 11 months ago

This now needs to be updated to nan 2.18.0 to support Node 21