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

Update troubleshooting.md #752

Merged
merged 1 commit into from
May 16, 2022
Merged

Update troubleshooting.md #752

merged 1 commit into from
May 16, 2022

Conversation

thonkinator
Copy link
Contributor

As I just learned, having spaces in the path of the directory which you are running npm i better-sqlite3 can cause issues with installation, presumably because, according to the error message, it does not insert escapes for characters like spaces. I just wasted two days, and I don't want anyone else to do so.

As I just learned, having spaces in the path of the directory which you are running `npm i better-sqlite3` can cause issues with installation, presumably because, according to the error message, it does not insert escapes for characters like spaces.
@Prinzhorn
Copy link
Contributor

I guess it won't hurt having it in there, multiple issues had been opened in the past because of that (e.g #547). But maybe make it a sentence at the end of the node-gyp item, because that's a limitation of node-gyp and not better-sqlite3.

@mceachen mceachen requested review from mceachen and removed request for mceachen March 11, 2022 18:25
@mceachen mceachen merged commit f639d61 into WiseLibs:master May 16, 2022
@mceachen
Copy link
Member

I added @Prinzhorn's suggestion to #814

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

Successfully merging this pull request may close these issues.

3 participants