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

Running pnpm run build in Windows fails #9328

Closed
7 tasks done
userquin opened this issue Jul 23, 2022 · 2 comments · Fixed by #9329
Closed
7 tasks done

Running pnpm run build in Windows fails #9328

userquin opened this issue Jul 23, 2022 · 2 comments · Fixed by #9329

Comments

@userquin
Copy link
Contributor

Describe the bug

Trying building Vite repo fails patching dist CJS:
imagen

Reproduction

run pnpm run build in this repo

System Info

System:
    OS: Windows 10 10.0.19044
    CPU: (16) x64 Intel(R) Core(TM) i9-9900K CPU @ 3.60GHz
    Memory: 39.14 GB / 63.95 GB
  Binaries:
    Node: 16.13.0 - C:\Program Files\nodejs\node.EXE
    npm: 8.1.0 - C:\Program Files\nodejs\npm.CMD
  Browsers:
    Chrome: 103.0.5060.134
    Edge: Spartan (44.19041.1266.0), Chromium (103.0.1264.62)
    Internet Explorer: 11.0.19041.1566

Used Package Manager

pnpm

Logs

Click to expand!
F:\work\projects\quini\GitHub\vite-new-hooks-symlinks\vite\packages\plugin-legacy:
 ERR_PNPM_RECURSIVE_RUN_FIRST_FAIL  @vitejs/plugin-legacy@2.0.0 build: `unbuild && pnpm run patch-cjs`
Exit status 1
 ELIFECYCLE  Command failed with exit code 1.

Validations

@bluwy
Copy link
Member

bluwy commented Jul 24, 2022

I got this before and is fixed after upgrading nodejs. I believe it was a bug.

@sodatea
Copy link
Member

sodatea commented Jul 24, 2022

Yeah, it's a bug in Node.js. nodejs/node#41388
Fixed in Node.js ^14.19, ^16.14, ^18.0

@github-actions github-actions bot locked and limited conversation to collaborators Aug 8, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants