Skip to content
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.

Sign up for GitHub

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

Jump to bottom

Replace ffi-napi with koffi to support Electron #1652

Open
ChaselDutt opened this issue Nov 6, 2024 · 2 comments
Open

Replace ffi-napi with koffi to support Electron #1652

ChaselDutt opened this issue Nov 6, 2024 · 2 comments

Comments

@ChaselDutt
Copy link

Error with Electron and ffi-napi #238
@nshmyrev
Copy link
Collaborator

Great idea, ff-napi is doomed. Let me check koffi!

@Boorj
Copy link

Boorj commented Jan 21, 2025 •
edited
Loading

I have implemented the workaround solution for my own purposes using ffi-rs (not koffi). Here's the gist.

Should I make a PR?

I haven't tested it with Electron, and also I have some concerns about this call: libvosk.vosk_recognizer_accept_waveform.async. But for the rest, it works perfectly in Node.js 22 without any node-gyp issues

Related issues:

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Milestone
No milestone
Development

No branches or pull requests
3 participants
@Boorj @nshmyrev @ChaselDutt