deno: 2.3.5 -> 2.3.6, fix tests#416177
Conversation
f86a597 to
4832a89
Compare
4832a89 to
663c2b6
Compare
|
A huge number of tests are failing on Linux (not on Darwin). I'm investigating |
6f18565 to
faa9fe3
Compare
faa9fe3 to
876eab6
Compare
|
I figured out the test crashes, the root cause is denoland/deno#29470. A prebuilt esbuild binary is now required for around half the tests, so I set that up in Also addressed the @06kellyjac @limwa I would appreciate your review and tests |
|
Tested on all 4 platforms and it built fine. |
|
Tested on x86-linux by building quarto (which depends on deno), and this sloves it! |
|
|
for the failure logs:
To be clear these builds ran on my arch machine but it has sandbox enabled & build agents. Running Probably easiest to skip all vsock related tests.
I may be being stupid here but I can't for the life of me find where the "denort" derivation comes from..... |
|
|
Seems like some tests are still flaky, maybe they should be disabled altogether, like before 1161ba5 ? |
Is this already happening on |
|
Looks like a regression |
I’ve had issues on master as well (it’s the reason I’m here), see: https://hydra.nixos.org/build/300144659#tabs-summary |
|
Let's just skip the flaky tests if flakiness is indeed the culprit. |
|
|
|
One downside of reenabing unit tests on For safety, I think the unit tests can remain disabled entirely in |
|
Found the |
cc59cdd to
9612e43
Compare
|
Yes, rebased the PR and I can reproduce on both aarch64 and x86_64 Linux :/ I'll try a git bisect |
|
|
|
That'll be wonderful to debug 😄 |
pkgs/by-name/de/deno/package.nix
Outdated
This comment was marked as outdated.
This comment was marked as outdated.
Sorry, something went wrong.
This comment was marked as outdated.
This comment was marked as outdated.
Sorry, something went wrong.
|
I'm pretty sure we hit this bug in the recently upgraded Rust + LLVM combination: rust-lang/rust#141737 The good news is disabling LTO fixes the build. |
|
|
Great work! I'd suggest using the new |
|
Oh, and #417619 can be closed, I think |
6e0a173 to
4c68c44
Compare
|
Yup, just wanted to finish testing on |
4c68c44 to
c0db458
Compare
|
Can you please format the code with |
c0db458 to
abc9b18
Compare
|
Done. I ran a nixpkgs-review with the 2 Darwin platforms and noticed that x86_64-darwin also crashes at linking (with a similar error). So I'm disabling LTO on x86_64-darwin and run a nixpkgs-review soon |
|
|
GaetanLepage
left a comment
There was a problem hiding this comment.
Thanks!
As deno is not building on x86_64-darwin on master, let's merge this PR and deal with gleam afterwards.
Nit (for next time): Try to keep comments out of Nix strings. It allows changing them without leading to useless rebuilds.
PR NixOS/nixpkgs#416177 was merged.
https://github.com/denoland/deno/releases/tag/v2.3.6
Running the tests have become more complicated since denoland/deno#29470 because a prebuilt
esbuildbinary is now required for around half the tests.Also addresses the crashes reported at #417331. I hope it's okay not to split the changes into multiple commits, my thinking was that the version upgrade is not buildable on all platforms without the test-related fixes.
Things done
nix.conf? (See Nix manual)sandbox = relaxedsandbox = truenix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage./result/bin/)Add a 👍 reaction to pull requests you find important.