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.

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

Installing node package on windows - incorrect configuration? #114

Open
mpdaugherty opened this issue Oct 8, 2024 · 5 comments
Open

Installing node package on windows - incorrect configuration? #114

mpdaugherty opened this issue Oct 8, 2024 · 5 comments

Comments

@mpdaugherty
Copy link

mpdaugherty commented Oct 8, 2024

I'm attempting to install the sqlite-vec node package on windows, but it appears that the optional dependency sqlite-vec-windows-x64 is configured to require os "windows" vs. "win32", so it won't install.

PS C:\Users\Michael\Documents\TestCode> npm i sqlite-vec-windows-x64
Debugger attached.
npm ERR! code EBADPLATFORM
npm ERR! notsup Unsupported platform for [email protected]: wanted {"os":"windows","cpu":"x64"} (current: {"os":"win32","cpu":"x64"})
npm ERR! notsup Valid os:   windows
npm ERR! notsup Actual os:  win32
npm ERR! notsup Valid cpu:  x64
npm ERR! notsup Actual cpu: x64

I don't know how the sqlite-dist.toml file is converted into npm packages, so I'm unable to create a PR for a fix unfortunately.

@mpdaugherty
Copy link
Author

I think we may end up rewriting the node code from the node package so we can specify the location of the .node file explicitly and install it ourselves. It would be nicer to use an official package, however, which would stay up to date with the latest releases.

@asg017
Copy link
Owner

asg017 commented Oct 11, 2024

Hey @mpdaugherty, thanks for the report - can you try with the latest 0.1.4-alpha.2 version and let me know if it works?

The os field is now win32, which should solve this issue. If so I'll release a 0.1.4 with the same fix

@mpdaugherty
Copy link
Author

Yes, thanks! I'll take a look, ideally can find some time today.

@mpdaugherty
Copy link
Author

@asg017 I can confirm that installed without a problem:

PS C:\Users\Michael\Documents\TestCode> npm i [email protected]

changed 1 package, and audited 3 packages in 2s

@asg017
Copy link
Owner

asg017 commented Oct 14, 2024

Thank you @mpdaugherty ! Will push out 0.1.4 shortly and will close this issue once it's out

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants