[L] Change in dart/sdk[main]: dart2wasm: Prepare standalone target

0 views
Skip to first unread message

Simon Binder (Gerrit)

unread,
Mar 7, 2026, 10:29:29 AM (3 days ago) Mar 7
to Martin Kustermann, dart-fe-te...@google.com, dart-uxr...@google.com, dart2wasm-t...@google.com, rev...@dartlang.org
Attention needed from Martin Kustermann

Simon Binder added 1 comment

Patchset-level comments
File-level comment, Patchset 1 (Latest):
Simon Binder . unresolved

I'm going to need some help configuring the test bots for this one, I've mostly copied test configurations through pattern matching. I've tested some locally, but I don't know if the CI bot is configured correctly.

Open in Gerrit

Related details

Attention is currently required from:
  • Martin Kustermann
Submit Requirements:
  • requirement is not satisfiedCode-Owners
  • requirement is not satisfiedCode-Review
  • requirement is not satisfiedReview-Enforcement
Inspect html for hidden footers to help with email filtering. To unsubscribe visit settings. DiffyGerrit
Gerrit-MessageType: comment
Gerrit-Project: sdk
Gerrit-Branch: main
Gerrit-Change-Id: I3f406afbf2dab65506094de5c3f4067f4db66f3e
Gerrit-Change-Number: 486380
Gerrit-PatchSet: 1
Gerrit-Owner: Simon Binder <o...@simonbinder.eu>
Gerrit-Reviewer: Martin Kustermann <kuste...@google.com>
Gerrit-Attention: Martin Kustermann <kuste...@google.com>
Gerrit-Comment-Date: Sat, 07 Mar 2026 15:29:23 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: No
unsatisfied_requirement
open
diffy

Simon Binder (Gerrit)

unread,
Mar 7, 2026, 2:35:26 PM (3 days ago) Mar 7
to Commit Queue, Kevin Moore, Martin Kustermann, dart-fe-te...@google.com, dart-uxr...@google.com, dart2wasm-t...@google.com, rev...@dartlang.org
Attention needed from Martin Kustermann

Simon Binder added 1 comment

Patchset-level comments
File-level comment, Patchset 3 (Latest):
Simon Binder . unresolved

I've removed the shell (d8, jsc, jsshell) tests to avoid having to specify runtime definitions twice.

Also it looks like the `dart2wasm-linux-standalone-chrome` builder wasn't part of the initial CQ dryrun, but I don't know what configuration might be missing for that.

Open in Gerrit

Related details

Attention is currently required from:
  • Martin Kustermann
Submit Requirements:
  • requirement is not satisfiedCode-Owners
  • requirement is not satisfiedCode-Review
  • requirement is not satisfiedReview-Enforcement
Inspect html for hidden footers to help with email filtering. To unsubscribe visit settings. DiffyGerrit
Gerrit-MessageType: comment
Gerrit-Project: sdk
Gerrit-Branch: main
Gerrit-Change-Id: I3f406afbf2dab65506094de5c3f4067f4db66f3e
Gerrit-Change-Number: 486380
Gerrit-PatchSet: 3
Gerrit-Owner: Simon Binder <o...@simonbinder.eu>
Gerrit-Reviewer: Kevin Moore <kev...@google.com>
Gerrit-Reviewer: Martin Kustermann <kuste...@google.com>
Gerrit-Attention: Martin Kustermann <kuste...@google.com>
Gerrit-Comment-Date: Sat, 07 Mar 2026 19:35:21 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: No
unsatisfied_requirement
open
diffy
Reply all
Reply to author
Forward
0 new messages