-
Notifications
You must be signed in to change notification settings - Fork 518
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
Prebuild binaries for Node 12 LTS (v72) #721
Comments
It's because Node 12 was released very recently and I haven't got the time to generate a prebuilt binary unfortunately |
I know, don't worry, it's nothing more than a reminder ;) |
Same problem, i am waiting too =) |
Any idea when it'll be ready to use? |
@CherryNerd You can use it, just install dependencies for source compile. A cyclone hit my city and currently, I am stuck at a place with no power and really poor internet, I cannot promise any ETA about when I would be able to provide a pre-compiled binary for Node 12. |
@agathver I hope you are well |
@frlinw can you write how to build it manually for the latest Node 12, I start a project with electron 5 and it only working with node 12 =( |
@tmirun it's in the README: install dependencies then update bcrypt and it will try to build it https://github.com/kelektiv/node.bcrypt.js#dependencies (Node 12 installer should ask you if you want to install node-gyp and its dependencies) |
@frlinw Oh, right, ty guy I will try it =) |
Bcrypt fails on Node v12 and there are related open issues for same kelektiv/node.bcrypt.js#728 and kelektiv/node.bcrypt.js#721. Let's wait for them to get fixed
Will this be fixed? |
Nodejs v12 is promoted to be LTS. I think it is time for bcrypt should support v12 |
ping? |
Why was this closed? |
Currently, bcrypt requires a manual build to work with the latest Node 12
The text was updated successfully, but these errors were encountered: