Update for Node.js 12.x support

[tomhughes]

No description provided.

[ShirasawaSama]

Cannot be used on electron.

Main thread is suspend. (It looks like an endless loop)

Whether it's in Mian Process, Render Process or WebWorker Process.

Code:

const Zip = require('zipfile')

Verions

• Electron: 6.0.9
• Nodejs: 12.4.0
• v8: 7.6.303.31-electron.0
• modules: 73
• uv: 1.29.1
• napi: 4
• chrome: 76.0.3809.146

Build

Environment

• Windows Server 2016
• Nodejs v12.12.0
• NPM v6.11.3

Install

.npmrc:

target=6.0.9
runtime=electron
build_from_source=true

Then:

npm i https://github.com/tomhughes/node-zipfile#node12

I really want to use this module, so I hope you can help me fix this bug. Thanks!

[tomhughes]

@ShirasawaSama you should open a separate issue for your problem rather than commenting on an unrelated pull request

[ShirasawaSama]

@tomhughes I'm sorry.

However, at present, this PR has not been merged (at least four months), so the current version of zipfile cannot be successfully built on node12.

Therefore, the environment is not the same as that of the original project.

Of course, I would like to thank you very much if you are willing to work with the project leader to fix this bug.