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

Node v20.2.0 create error #238

Closed
sergical opened this issue Jun 27, 2023 · 3 comments
Closed

Node v20.2.0 create error #238

sergical opened this issue Jun 27, 2023 · 3 comments

Comments

@sergical
Copy link

pnpm create ponder
../Library/pnpm/store/v3/tmp/dlx-13731   |  +39 ++++
Packages are hard linked from the content-addressable store to the virtual store.
  Content-addressable store is at: /Users/sergiydybskiy/Library/pnpm/store/v3
  Virtual store is at:             ../Library/pnpm/store/v3/tmp/dlx-13731/node_modules/.pnpm
../Library/pnpm/store/v3/tmp/dlx-13731   | Progress: resolved 39, reused 39, downloaded 0, added 39, done
✔ What is your project named? … bera-graph-test
✔ Would you like to use a template for this project? › None

Creating a new Ponder app in /Users/sergiydybskiy/src/bera-graph-test.

Installing with pnpm.
node:internal/errors:862
  const err = new Error(message);
              ^

Error: Command failed: pnpm --silent install
    at checkExecSyncError (node:child_process:885:11)
    at execSync (node:child_process:957:15)
    at run (/Users/sergiydybskiy/Library/pnpm/store/v3/tmp/dlx-13731/node_modules/.pnpm/[email protected]/node_modules/create-ponder/dist/create-ponder.js:786:42)
    at process.processTicksAndRejections (node:internal/process/task_queues:95:5)
    at async createPonder (/Users/sergiydybskiy/Library/pnpm/store/v3/tmp/dlx-13731/node_modules/.pnpm/[email protected]/node_modules/create-ponder/dist/create-ponder.js:928:3) {
  status: 1,
  signal: null,
  output: [ null, null, null ],
  pid: 13745,
  stdout: null,
  stderr: null
}

Node.js v20.2.0
 ERROR  Command failed with exit code 1: create-ponder

pnpm: Command failed with exit code 1: create-ponder
    at makeError (/Users/sergiydybskiy/Library/pnpm/global/5/.pnpm/[email protected]/node_modules/pnpm/dist/pnpm.cjs:24241:17)
    at handlePromise (/Users/sergiydybskiy/Library/pnpm/global/5/.pnpm/[email protected]/node_modules/pnpm/dist/pnpm.cjs:24812:33)
    at process.processTicksAndRejections (node:internal/process/task_queues:95:5)
    at async Object.handler (/Users/sergiydybskiy/Library/pnpm/global/5/.pnpm/[email protected]/node_modules/pnpm/dist/pnpm.cjs:207250:7)
    at async /Users/sergiydybskiy/Library/pnpm/global/5/.pnpm/[email protected]/node_modules/pnpm/dist/pnpm.cjs:216656:21
    at async main (/Users/sergiydybskiy/Library/pnpm/global/5/.pnpm/[email protected]/node_modules/pnpm/dist/pnpm.cjs:216623:34)
    at async runPnpm (/Users/sergiydybskiy/Library/pnpm/global/5/.pnpm/[email protected]/node_modules/pnpm/dist/pnpm.cjs:216878:5)
    at async /Users/sergiydybskiy/Library/pnpm/global/5/.pnpm/[email protected]/node_modules/pnpm/dist/pnpm.cjs:216870:7
@typedarray
Copy link
Collaborator

typedarray commented Jun 27, 2023

Thanks for reporting! I was able to reproduce this locally by running pnpm create ponder while using Node 20 (nvm use 20).

The problem is related to installing the dependency better-sqlite3. This library is unfortunately riddled with installation issues, I've personally spent hours on this. I opened a new issue here WiseLibs/better-sqlite3#1027, hoping to get some clarity there so I can mitigate.

@anudit
Copy link
Contributor

anudit commented Jul 17, 2023

Hey @0xOlias looks like they merged support for Node v20 in 8.4.0 Could we get a version update on Ponder?

@typedarray
Copy link
Collaborator

Fixed in #292, released in 0.0.79.

One small trade-off is that folks using pnpm may now need to install node-gyp globally before running pnpm create ponder (regardless of Node version).

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

3 participants