frida / frida-node

Frida Node.js bindings
282 stars 66 forks source link

Compile support for Node.js v12 #39

Closed gebing closed 5 years ago

gebing commented 5 years ago

Pull request for Issues #38, add compile support for Node.js v12

oleavr commented 5 years ago

Thanks! Could we use nan here instead to make these bits portable? (We are already using it for other parts.)

oleavr commented 5 years ago

Needed this myself and had to get a release out quick, so I took a slightly different approach in 2328c51780670b6041f12e337c3738262be02abe. Currently releasing Frida 12.5.6 with that change included. Cheers!