search

WiseLibs / better-sqlite3

The fastest and simplest library for SQLite3 in Node.js.
MIT License
5.56k stars 393 forks source link

N-API Support #271

Open Flackus opened 5 years ago

Flackus commented 5 years ago

Grretings.

First of all, thanks for all the effort creating and maintaining this module, it's very efficient and convenient to use.

I would like to know if there are any plans for supporting N-API https://nodejs.org/docs/latest-v8.x/api/n-api.html?

Possibly related issue with a lot of info in it – https://github.com/mapbox/node-sqlite3/issues/830

zackster commented 4 years ago

Anyone care to provide an outline of what would be required to implement support for N-API?

zoubingwu commented 3 years ago

Any plans on this? It gonna save tons of time doing re-compilation especially for electron apps.

tessro commented 3 years ago

@JoshuaWise would you be open to a PR that ported the V8 interface to N-API? I've been noodling on the continuous maintenance needed for Electron (#638, #693, etc.) and I believe using N-API might alleviate the need to update the build scripts every release.

tessro commented 3 years ago

Bumping this. I'm interested in taking this on, although I believe it's worthwhile porting off lzz onto vanilla C++ as part of this change. lzz is a learning barrier to contributors and is no longer available for download.

mohd-akram commented 2 years ago

Any update on this?