-
Notifications
You must be signed in to change notification settings - Fork 3.4k
Fix import.meta.dirname in node 18 #23407
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
Conversation
No, you are not doing anything wrong. This was an unfortunate side effect of #23349. I accidentally caused the compiler to start requiring node v19. See #23396 emsdk was updated to v20 a while back now so it went unnoticed. How did you end up with v18 on your system? Is it an old emsdk version maybe? Maybe you also forgot to do |
This is just the system node. I have it set to be used in my This does seem to fix things for me. Can we land it, or does it have downsides? |
The right way to revert this would be to restore this part of the code to how it was before #23349. |
Ok, I'll switch to 20 locally then. |
No, we should still support v18 if its the one in debian/stable. Working on anther change now.. |
In emscripten-core#23349 we (I) accidentally broke running emscripten with node older than v19 and as it happens debian/stable still ships v18: https://packages.debian.org/bookworm/nodejs This change reverts emscripten-core#23349 and also explicitly bumps the minimum version up to v18.20. I've also added some testing to CI to ensure we can actually run using this version. See emscripten-core#23396 and emscripten-core#23407
In emscripten-core#23349 we (I) accidentally broke running emscripten with node older than v19 and as it happens debian/stable still ships v18: https://packages.debian.org/bookworm/nodejs This change reverts emscripten-core#23349 and also explicitly bumps the minimum version up to v18.20. I've also added some testing to CI to ensure we can actually run using this version. See emscripten-core#23396 and emscripten-core#23407
In emscripten-core#23349 we (I) accidentally broke running emscripten with node older than v19 and as it happens debian/stable still ships v18: https://packages.debian.org/bookworm/nodejs This change reverts emscripten-core#23349 and also explicitly bumps the minimum version up to v18.20. I've also added some testing to CI to ensure we can actually run using this version. See emscripten-core#23396 and emscripten-core#23407
In emscripten-core#23349 we (I) accidentally broke running emscripten with node older than v19 and as it happens debian/stable still ships v18: https://packages.debian.org/bookworm/nodejs This change reverts emscripten-core#23349 and also explicitly bumps the minimum version up to v18.20. I've also added some testing to CI to ensure we can actually run using this version. See emscripten-core#23396 and emscripten-core#23407
In emscripten-core#23349 we (I) accidentally broke running emscripten with node older than v19 and as it happens debian/stable still ships v18: https://packages.debian.org/bookworm/nodejs This change reverts emscripten-core#23349 and also explicitly bumps the minimum version up to v18.20. I've also added some testing to CI to ensure we can actually run using this version. See emscripten-core#23396 and emscripten-core#23407
In emscripten-core#23349 we (I) accidentally broke running emscripten with node older than v19 and as it happens debian/stable still ships v18: https://packages.debian.org/bookworm/nodejs This change reverts emscripten-core#23349 and also explicitly bumps the minimum version up to v18. I've also added some testing to CI to ensure we can actually run using this version. See emscripten-core#23396 and emscripten-core#23407
In emscripten-core#23349 we (I) accidentally broke running emscripten with node older than v19 and as it happens debian/stable still ships v18: https://packages.debian.org/bookworm/nodejs This change reverts emscripten-core#23349 and also explicitly bumps the minimum version up to v18. I've also added some testing to CI to ensure we can actually run using this version. See emscripten-core#23396 and emscripten-core#23407
In emscripten-core#23349 we (I) accidentally broke running emscripten with node older than v19 and as it happens debian/stable still ships v18: https://packages.debian.org/bookworm/nodejs This change reverts emscripten-core#23349 and also explicitly bumps the minimum version up to v18. I've also added some testing to CI to ensure we can actually run using this version. See emscripten-core#23396 and emscripten-core#23407
In emscripten-core#23349 we (I) accidentally broke running emscripten with node older than v19 and as it happens debian/stable still ships v18: https://packages.debian.org/bookworm/nodejs This change reverts emscripten-core#23349 and also explicitly bumps the minimum version up to v18. I've also added some testing to CI to ensure we can actually run using this version. See emscripten-core#23396 and emscripten-core#23407
In #23349 we (I) accidentally broke running emscripten with node older than v19 and as it happens debian/stable still ships v18: https://packages.debian.org/bookworm/nodejs This change reverts #23349 and also explicitly bumps the minimum version up to v18. I've also added some testing to CI to ensure we can actually run using this version. See #23396 and #23407
Likely I am doing something wrong, but I need this change, or else this happens for me locally with node
v18.20.1
:Clearing the cache, running bootstrap,
npm ci
etc., nothing works except for this patch.