-
Notifications
You must be signed in to change notification settings - Fork 44
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
Unable to build or preview on Windows with pnpm #225
Comments
Have you tried removing node_modules and running |
I have. I'm just trying to do a complete fresh install now. I asked some colleagues to try the same thing and it worked for them, so potentially something is broken with my setup. Will update if it fixes anything. |
Issue remains. Have reinstalled pnpm. On the latest version 9.12.2. Have made sure long paths on Windows are enabled. Etc. |
Thanks for trying all of that. I fired up a VM on my end and couldn't reproduce it. I'm going to add some new CLI tests for Ubuntu and Windows to the CI/CD workflows, but without being able to reproduce it, it's going to be very difficult to find a resolution. There's a few other things you can try to self-traige:
This does look like a dependency resolution issue, so you might also try using a different node-modules linker strategy https://pnpm.io/npmrc#hoist
|
Thanks for trying. Here are some other things I tried and the results, in case something helps. Using it without install did not work.
Adding Using |
That's really interesting. It's definitely a If you're working from a branch or a fork, it may be worth running Also worth trying is the Next step, if none of the above work, I'd recommend is completely nuking your local store (which I've also had to do a few times). First thing I do is I could also take a look at your lockfile to see if there's anything which stands out there, if you'd like to put that into a gist (https://gist.github.com) |
Dont worry. It was Windows all along 😭 I was demonstrating the issue to someone and it worked (observer effect...). After a bit more playing around I tried to make sure it wasn't the path issue, and lo and behold, at 261 characters Node broke. The specific path that broke was: I then found the solution: nodejs/node#50753 (comment) Upgrading to Node 22.7 fixed the issue. Massively massively frustrating issue... but thank you very much for the help. |
No problem! Very happy you were able to find the issue. I'll make a mental note about the Node version as well. Really appreciate your time triaging and sharing what the issue ended up being. |
Actually, that solution seemed to not work. I tried with a longer base path and have started getting the same issues as before. This is even with |
Definitely open an issue with pnpm about this. I'm not sure there's anything actionable that I can do with this project. Is using WSL an option? My guess is that you'll run into this again with any long dependency paths |
This is definitely a Node / pnpm issue. Just wanted to inform you. I'll open an issue with them in the morning. Unfortunately, WSL is a last resort, but I tested the same thing there and it worked. Thank you again for being so helpful and looking into this! For posterity, I was able to get my specific situation to start working by setting |
@jsxp-email/cli
Version?: 2.0.11Expected Behavior
On Windows, using
pnpm add jsx-email
and thenpnpm exec email
should work.Actual Behavior
I get the following:
Additional Information
I've tried to clear the pnpm store, clear cache, reinstall, etc. Still the same issue.
I tried running the same command on WSL and it works fine.
The text was updated successfully, but these errors were encountered: