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

Dependency resolution broken #6986

Closed
M-Gonzalo opened this issue Nov 8, 2023 · 9 comments
Closed

Dependency resolution broken #6986

M-Gonzalo opened this issue Nov 8, 2023 · 9 comments
Labels
bug Something isn't working bun install Something that relates to the npm-compatible client needs investigate Needs to be investigated to find the root cause

Comments

@M-Gonzalo
Copy link

What version of Bun is running?

1.0.10+d85bd49d8

What platform is your computer?

Linux 6.5.9-zen2-1-zen x86_64 unknown

What steps can reproduce the bug?

I'm trying to run an express server that works perfectly with nodejs, but I get a error: FileNotFound reading "<project_path>/node_modules/.pnpm/side-channel@1.0.4/node_modules/get-intrinsic"

What is the expected behavior?

No response

What do you see instead?

No response

Additional information

This is NOT a bug related to bun install. My command is bun server.js

@M-Gonzalo M-Gonzalo added the bug Something isn't working label Nov 8, 2023
@Jarred-Sumner
Copy link
Collaborator

Can you test if this works in v1.0.7? I wonder if this is a regression from the Windows changes. It touched a lot of error handling code and if we incorrectly handle symlink resolution that would cause this.

@M-Gonzalo
Copy link
Author

Can you test if this works in v1.0.7? I wonder if this is a regression from the Windows changes. It touched a lot of error handling code and if we incorrectly handle symlink resolution that would cause this.

I did. Same problem:
image
Someone suggested it could be the same as #4959 but I tried the workaround and it didn't change anything either

@Jarred-Sumner
Copy link
Collaborator

Hmm, what if you try a much older version of Bun, like v0.6.1? curl https://bun.sh/install | bash -s “bun-v0.6.1”

@M-Gonzalo
Copy link
Author

Hmm, what if you try a much older version of Bun, like v0.6.1? curl https://bun.sh/install | bash -s “bun-v0.6.1”

Yes, 0.6.1 got past the dependency bug:
Screenshot_20231108_102021

BTW, the download script gave me a 404:
image

@Jarred-Sumner
Copy link
Collaborator

Okay very interesting. I wonder what version this regressed in.

@M-Gonzalo
Copy link
Author

Okay very interesting. I wonder what version this regressed in.

I can try and suss it out. Any chance you have a zip with all versions together or something like that? It'll be much faster than browse github releases for each one

@Jarred-Sumner
Copy link
Collaborator

It might be without the bun- prefix

try

curl https://bun.sh/install | bash -s “v0.6.1”

@M-Gonzalo
Copy link
Author

It might be without the bun- prefix

try

curl bun.sh/install | bash -s “v0.6.1”

With single quotes it works


FOUND IT! v0.6.10 is the first time I see the FileNotFound error

@Electroid Electroid added bun install Something that relates to the npm-compatible client needs investigate Needs to be investigated to find the root cause labels Nov 8, 2023
@M-Gonzalo
Copy link
Author

I don't know if it's fixed already, but on 1.0.12 I'm not seeing this error, but another one related to node_module_register and canvas (#4290 (comment))

It could be that this other error presents itself before the one in the present issue, or that this one here is fixed. I get a feeling that it's the latter because this error here used to appear almost immediately, but the error about canvas takes a whole second to appear.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working bun install Something that relates to the npm-compatible client needs investigate Needs to be investigated to find the root cause
Projects
None yet
Development

No branches or pull requests

3 participants