auto label is removed for flutter/engine/46842, due to - The status or check suite Linux linux_web_engine has failed. Please fix the issues identified (or deflake) before re-applying this label.

I had started looking at this, and IIUC the packages parameter in dart_action will add a --packages flag to the Dart command line (making it redundant with packages_args.

So I'd suggest:

• remove packages_args
• for the prebuilt Dart SDK case, add the packages file to inputs and the --packages flag to args if package_config is defined
• for the non-prebuilt case, set the packages parameter in dart_action

Also, after applying the fix I'm still seeing errors when configuring an Android build with --no-prebuilt-dart-sdk:

ERROR Unresolved dependencies.
//flutter/flutter_frontend_server:frontend_server(//build/toolchain/android:clang_arm)
  needs //flutter/build/dart:dart_sdk(//build/toolchain/android:clang_arm)

@jason-simmons Good catch. I've updated the PR.

With this patch I'm still seeing errors when building a 32-bit android_debug_unopt target on an x64 host.

When it builds the scenario app snapshot, the frontend server fails with:

output: VM initialization failed: JIT cannot simulate target architecture with different word size than host

It turns out that when not using the prebuilt Dart SDK, we were previously running the frontend server from source. I've updated this patch to go back to that behavior. Thanks, Jason!

Golden file changes have been found for this pull request. Click here to view and triage (e.g. because this is an intentional change).

If you are still iterating on this change and are not ready to resolve the images on the Flutter Gold dashboard, consider marking this PR as a draft pull request above. You will still be able to view image results on the dashboard, commenting will be silenced, and the check will not try to resolve itself until marked ready for review.

Changes reported for pull request #46842 at sha 971c37b

Will merge this into the reland of #46863