[ Tool ] Add support for linux riscv64 architecture by vhaudiquet · Pull Request #178711 · flutter/flutter

vhaudiquet · 2025-11-18T09:25:01Z

This adds support for buildind desktop linux applications for riscv64 on the flutter tool, as well as basic riscv64 support for the tool.

This is a first move towards fixing #99963. Along with #178712 that allows building an engine for riscv64, this PR allows me to build and run the flutter 'hello world' app on RISC-V hardware:

This allows someone who built a flutter engine and exposed it on a webserver to run commands like:

export FLUTTER_STORAGE_BASE_URL=http://your-server-url
flutter
flutter doctor
cd example/hello_world && flutter build linux

This means that even if Google does not want to support Flutter on RISC-V officially (hosting the artifacts), community-managed servers could host the RISC-V flutter engines/artifacts. It also means that internally, companies using Flutter on RISC-V would just have to build their engine and setup such a server, and it allows them to use the upstream Flutter tool.

Pre-launch Checklist

I read the Contributor Guide and followed the process outlined there for submitting PRs.
I read the Tree Hygiene wiki page, which explains my responsibilities.
I read and followed the Flutter Style Guide, including Features we expect every widget to implement.
I signed the CLA.
I listed at least one issue that this PR fixes in the description above.
I updated/added relevant documentation (doc comments with ///).
I added new tests to check the change I am making, or this PR is test-exempt.
I followed the breaking change policy and added Data Driven Fixes where supported.
All existing and new tests are passing.

If you need help, consider asking for advice on the #hackers-new channel on Discord.

Note: The Flutter team is currently trialing the use of Gemini Code Assist for GitHub. Comments from the gemini-code-assist bot should not be taken as authoritative feedback from the Flutter team. If you find its comments useful you can update your code accordingly, but if you are unsure or disagree with the feedback, please feel free to wait for a Flutter team member's review for guidance on which automated comments should be addressed.

gemini-code-assist

Code Review

This pull request adds support for the riscv64 architecture for desktop Linux applications in the Flutter tool. The changes are consistent across multiple files, adding the new architecture to enums, switch statements, and build configurations. My review focuses on improving code readability in one of the new logic blocks. Overall, the changes look good and are a solid step towards supporting Flutter on RISC-V.

vhaudiquet · 2025-11-19T18:16:42Z

I think my last modification fixed all the tests (formatting, and changing a test case expected output to include riscv64).

The last failing "Windows tool_tests_commands" test might be an infra issue, as it seems to have timed out waiting for a prompt... I'm not sure that it is related to my PR. It seems I can't trigger a retry myself.

@jtmcdole what do you think?

reidbaker

Approved but I would want someone from the engine team to take a look before this lands.

reidbaker

Not sure where you are on the process of landing this feature but I would have expected to see a test that built for risc-v as part of this pr to ensure we dont regress on the ability to build.

vhaudiquet · 2025-11-24T19:10:33Z

Thank you! About the testing part, I wasn't sure that the testing infrastructure had riscv64 support; and I would rather have untested RISC-V support than no RISC-V support at all.

If you confirm that I could write automatic tests that would run on riscv64, or enable riscv64 for existing tests, I can try!

reidbaker · 2025-11-24T19:15:23Z

At the minimum I would expect that you could author a test that attempted to build the risc-v code. That code would be required before we can publish an RISC-V engine binary.

vhaudiquet · 2025-11-25T12:55:41Z

I tried to understand the test infrastructure and add a test that is basically a copy of the one present on arm64, adapted for riscv64. Is that what you had in mind?

Edit: Again, the failed CI check seems to be a timeout unrelated to the test I added, on preview_detector_workspace_test.dart: PreviewDetector

reidbaker

Thank you for adding a build test. This should still be approved by someone on the engine team but I no longer see any issues.

gaaclarke

LGTM. @reidbaker the tests here assert that the tool will attempt to build riscv64 but we don't actually buildit as part of CI right? We should consider adding that as a follow up.

reidbaker · 2025-11-25T17:02:14Z

LGTM. @reidbaker the tests here assert that the tool will attempt to build riscv64 but we don't actually buildit as part of CI right? We should consider adding that as a follow up.

Correct, I am viewing this feature as in progress and this test makes sure we dont accidentally break support but it will require building the engine manually. I actually think if this pr added ci support for publishing risc-v we might not take it without leads approval.

vhaudiquet · 2025-11-25T17:08:45Z

I also submitted #178712 to allow building the engine (either locally or in CI). I started a discussion there, that is still ongoing it seems.

Having official tool support could still be useful even if you don't plan/want to build the engine on CI, as it would allow third-party actors to use the official tool with their third-party built engine using export FLUTTER_STORAGE_BASE_URL as described; this is why I submitted two different PRs.

filiph · 2025-11-25T17:11:37Z

Was I added in error, @reidbaker? I don't have much experience with this part of the codebase.

reidbaker · 2025-11-25T17:14:18Z

Was I added in error, @reidbaker? I don't have much experience with this part of the codebase.

Yes sorry! Removed.

loic-sharma · 2025-11-25T17:29:17Z

cc @robert-ancell as this change adds riscv64 support to Flutter Linux.
cc @dcharkes as this lightly touches native assets.

loic-sharma · 2025-11-25T17:34:51Z

Drive by comment: When we added Windows Arm64 artifacts we broke flutter precache --all --force on Windows (see #164431).

If you haven't already, please check that this command still works on Linux hosts.

This adds support for buildind desktop linux applications for riscv64 on the flutter tool, as well as basic riscv64 support for the tool.

vhaudiquet · 2025-12-02T20:00:26Z

The google testing is the only one failing now. Is that a real fail, or a temporary/infra thing?

zanderso

lgtm

A Googler will need to follow step 5 of https://github.com/flutter/flutter/blob/master/docs/contributing/Feature-flags.md. @bkonyi would you be able to help out with that 2 line change?

jtmcdole · 2026-01-08T22:06:47Z

The google testing is the only one failing now. Is that a real fail, or a temporary/infra thing?

This is a real failure from a custom implementation of the tool. Let me take a quick look.

jtmcdole · 2026-01-08T22:56:19Z

This PR is 700+ commits behind master; updating to pick up changes.

jtmcdole · 2026-01-09T00:09:55Z

Applying another fix internally

In #178711, `packages/flutter_tools/lib/src/flutter_cache.dart` was updated to add 'riscv' downloading in the `FlutterRunnerDebugSymbols`. This class is not well named and is responsible for downloading "the debug symbols for flutter runner for Fuchsia development." We do not produce fuchsia riscv. Fixes #180775

…flutter#178711)

Roll Flutter from 01d37bc to 73769a2 (65 revisions) flutter/flutter@01d37bc...73769a2 2026-01-10 [email protected] Roll Dart SDK from 5e855c2bb3ef to 87fbfd5381b6 (1 revision) (flutter/flutter#180800) 2026-01-10 [email protected] Roll Skia from b2b109f0e980 to f39cc645b1dd (2 revisions) (flutter/flutter#180796) 2026-01-10 [email protected] Roll Dart SDK from b7963905e6a2 to 5e855c2bb3ef (2 revisions) (flutter/flutter#180794) 2026-01-10 [email protected] Make sure that a CupertinoTabScaffold doesn't crash in 0x0 environment (flutter/flutter#179824) 2026-01-10 [email protected] Roll Dart SDK from d25ad331b7ea to b7963905e6a2 (2 revisions) (flutter/flutter#180783) 2026-01-10 [email protected] Make sure that a Container doesn't crash in 0x0 environment (flutter/flutter#180350) 2026-01-10 [email protected] Make sure that an Expansible doesn't crash in 0x0 environment (flutter/flutter#180478) 2026-01-10 98614782+auto-submit[bot]@users.noreply.github.com Reverts "Enabled some disabled impeller fragment shader dart tests (#180759)" (flutter/flutter#180785) 2026-01-09 [email protected] Merge `widget_tester_leaks_free_test.dart` into `widget_tester_test.dart` (flutter/flutter#180600) 2026-01-09 [email protected] fix: there are no riscv fuchsia artifacts (flutter/flutter#180779) 2026-01-09 [email protected] Roll Skia from 7386219151e6 to b2b109f0e980 (1 revision) (flutter/flutter#180771) 2026-01-09 [email protected] Re-prioritize pipeline compile jobs and perform them eagerly instead of waiting. (flutter/flutter#180022) 2026-01-09 [email protected] Enabled some disabled impeller fragment shader dart tests (flutter/flutter#180759) 2026-01-09 [email protected] Roll Fuchsia Linux SDK from rxeg-6UB678HKJ4UQ... to 83Favz_zzMzdVuOHg... (flutter/flutter#180765) 2026-01-09 [email protected] [A11y ] Add `clearSemantics`in table (flutter/flutter#180665) 2026-01-09 [email protected] Update CODEOWNERS to remove chinmaygarde. (flutter/flutter#180703) 2026-01-09 [email protected] [ Tool ] Add support for linux riscv64 architecture (flutter/flutter#178711) 2026-01-09 [email protected] Roll Skia from e9b3264ade0c to 7386219151e6 (12 revisions) (flutter/flutter#180754) 2026-01-09 [email protected] Roll Dart SDK from fe2ba2c5dd50 to d25ad331b7ea (10 revisions) (flutter/flutter#180741) 2026-01-09 98614782+auto-submit[bot]@users.noreply.github.com Reverts "Unpin google_mobile_ads (#180573)" (flutter/flutter#180761) 2026-01-09 [email protected] Fix typo in dropdown_menu.dart (flutter/flutter#180172) 2026-01-09 [email protected] Roll Packages from 039a026 to 51fe1d9 (1 revision) (flutter/flutter#180742) 2026-01-09 [email protected] Unpin google_mobile_ads (flutter/flutter#180573) 2026-01-09 [email protected] Update flutter changelog for 3.38.6 (flutter/flutter#180708) 2026-01-08 98614782+auto-submit[bot]@users.noreply.github.com Reverts "Fix iOS xattr removal to clear all extended attributes (#180355)" (flutter/flutter#180709) 2026-01-08 [email protected] Make sure that an EditableText doesn't crash in 0x0 environment (flutter/flutter#180457) 2026-01-08 [email protected] Implementation of tooltip windows for win32 (flutter/flutter#179147) 2026-01-08 [email protected] [web] Don't serve files outside of project (flutter/flutter#180699) 2026-01-08 [email protected] Roll Skia from 837be28dd218 to e9b3264ade0c (1 revision) (flutter/flutter#180702) 2026-01-08 [email protected] Add new motion accessibility features to iOS. (flutter/flutter#178102) 2026-01-08 [email protected] Fix iOS xattr removal to clear all extended attributes (flutter/flutter#180355) 2026-01-08 [email protected] [ Widget Preview ] Move widget_preview_scaffold tests to `dev/integration_tests/widget_preview_scaffold` (flutter/flutter#180658) 2026-01-08 [email protected] De-interleaves engine dart test output (flutter/flutter#180651) 2026-01-08 [email protected] [web] Fix SemanticsService.announce not working inside dialogs (flutter/flutter#179958) 2026-01-08 [email protected] Roll Skia from 42233226ac56 to 837be28dd218 (2 revisions) (flutter/flutter#180693) 2026-01-08 [email protected] Roll Dart SDK to 3.11.0-296.2.beta (flutter/flutter#180685) 2026-01-08 [email protected] Improve code quality in `AndroidTouchProcessorTest.java` (flutter/flutter#180583) 2026-01-08 [email protected] Revert "Reverts "[reland] Unify canvas creation and Surface code in S…kwasm and CanvasKit (#179473)" (#180152)" (flutter/flutter#180610) 2026-01-08 [email protected] Roll Skia from a0c407bce408 to 42233226ac56 (4 revisions) (flutter/flutter#180688) 2026-01-08 [email protected] [ Tool ] Fix flake in overall_experience_test.dart (flutter/flutter#180655) 2026-01-08 [email protected] Roll Packages from 9705815 to 039a026 (6 revisions) (flutter/flutter#180684) 2026-01-08 [email protected] flutter_tools: Auto-generate ExportOptions.plist for manual iOS code signing (flutter/flutter#177888) 2026-01-08 [email protected] Roll Skia from 58837e160874 to a0c407bce408 (2 revisions) (flutter/flutter#180679) 2026-01-08 [email protected] Roll Skia from 1e3266fdba86 to 58837e160874 (1 revision) (flutter/flutter#180677) 2026-01-08 [email protected] Roll Skia from 3c47ea10638f to 1e3266fdba86 (4 revisions) (flutter/flutter#180675) 2026-01-08 [email protected] Roll Fuchsia Linux SDK from dTvN_JVSCfGFRasvH... to rxeg-6UB678HKJ4UQ... (flutter/flutter#180673) ...

vhaudiquet requested review from a team and jtmcdole as code owners November 18, 2025 09:25

github-actions Bot added tool Affects the "flutter" command-line tool. See also t: labels. a: desktop Running on desktop team-android Owned by Android platform team labels Nov 18, 2025

gemini-code-assist Bot reviewed Nov 18, 2025

View reviewed changes

Comment thread packages/flutter_tools/lib/src/commands/build_linux.dart Outdated

vhaudiquet mentioned this pull request Nov 18, 2025

Add support for riscv64 desktop linux engine #178712

Open

5 of 9 tasks

vhaudiquet force-pushed the riscv64-tool branch 3 times, most recently from 8851c72 to 16fd221 Compare November 19, 2025 17:29

reidbaker approved these changes Nov 24, 2025

View reviewed changes

reidbaker approved these changes Nov 25, 2025

View reviewed changes

reidbaker requested a review from gaaclarke November 25, 2025 16:20

reidbaker changed the title ~~[ Tool ] Add support riscv64 architecture~~ [ Tool ] Add support for linux riscv64 architecture Nov 25, 2025

gaaclarke approved these changes Nov 25, 2025

View reviewed changes

reidbaker requested a review from filiph November 25, 2025 17:06

reidbaker removed the request for review from filiph November 25, 2025 17:14

loic-sharma requested a review from robert-ancell November 25, 2025 17:28

loic-sharma requested a review from bkonyi November 25, 2025 17:36

vhaudiquet force-pushed the riscv64-tool branch from 0e13adc to 38460c1 Compare December 2, 2025 17:47

zanderso reviewed Dec 2, 2025

View reviewed changes

Comment thread packages/flutter_tools/lib/src/commands/build_linux.dart

vhaudiquet added 2 commits December 2, 2025 19:52

[ Tool ] Support riscv64 architecture

40d0c71

This adds support for buildind desktop linux applications for riscv64 on the flutter tool, as well as basic riscv64 support for the tool.

[ Tool ] Add riscv64 tool/commands build tests

49d212d

vhaudiquet force-pushed the riscv64-tool branch from 38460c1 to 49d212d Compare December 2, 2025 18:55

zanderso approved these changes Dec 3, 2025

View reviewed changes

Kataglyphis mentioned this pull request Dec 12, 2025

Pipline is not succeeding ardera/flutter-ci#26

Closed

Merge branch 'master' into riscv64-tool

3b17cb2

jtmcdole added the autosubmit Merge PR when tree becomes green via auto submit App label Jan 9, 2026

auto-submit Bot added this pull request to the merge queue Jan 9, 2026

Merged via the queue into flutter:master with commit a3c12e6 Jan 9, 2026
148 checks passed

flutter-dashboard Bot removed the autosubmit Merge PR when tree becomes green via auto submit App label Jan 9, 2026

jtmcdole mentioned this pull request Jan 9, 2026

fix: there are no riscv fuchsia artifacts #180779

Merged

engine-flutter-autoroll mentioned this pull request Jan 10, 2026

Roll Flutter from 01d37bc80fd5 to 0b7596b3f269 (64 revisions) flutter/packages#10768

Closed

engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request Jan 10, 2026

a3c12e6 [ Tool ] Add support for linux riscv64 architecture (flutter/…

28443ad

…flutter#178711)

engine-flutter-autoroll mentioned this pull request Jan 10, 2026

Roll Flutter from 01d37bc80fd5 to 73769a220639 (65 revisions) flutter/packages#10769

Closed

engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request Jan 10, 2026

a3c12e6 [ Tool ] Add support for linux riscv64 architecture (flutter/…

dd4fc91

…flutter#178711)

engine-flutter-autoroll mentioned this pull request Jan 10, 2026

Roll Flutter from 01d37bc80fd5 to 73769a220639 (65 revisions) flutter/packages#10770

Merged

engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request Jan 10, 2026

a3c12e6 [ Tool ] Add support for linux riscv64 architecture (flutter/…

ef24beb

…flutter#178711)

vhaudiquet mentioned this pull request Jun 11, 2026

[BUG] RISC-V: Download the right dart SDK (and engine ?) for real RISC-V support leoafarias/fvm#969

Closed

1 task done

MichaIng mentioned this pull request Jul 4, 2026

Add riscv64 Support rustdesk/rustdesk-server#452

Open

Conversation

vhaudiquet commented Nov 18, 2025 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Pre-launch Checklist

Uh oh!

gemini-code-assist Bot left a comment

Choose a reason for hiding this comment

Code Review

Uh oh!

Uh oh!

vhaudiquet commented Nov 19, 2025 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

reidbaker left a comment

Choose a reason for hiding this comment

Uh oh!

reidbaker left a comment

Choose a reason for hiding this comment

Uh oh!

vhaudiquet commented Nov 24, 2025

Uh oh!

reidbaker commented Nov 24, 2025

Uh oh!

vhaudiquet commented Nov 25, 2025 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

reidbaker left a comment

Choose a reason for hiding this comment

Uh oh!

gaaclarke left a comment

Choose a reason for hiding this comment

Uh oh!

reidbaker commented Nov 25, 2025

Uh oh!

vhaudiquet commented Nov 25, 2025 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

filiph commented Nov 25, 2025

Uh oh!

reidbaker commented Nov 25, 2025

Uh oh!

loic-sharma commented Nov 25, 2025 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

loic-sharma commented Nov 25, 2025

Uh oh!

Uh oh!

vhaudiquet commented Dec 2, 2025

Uh oh!

zanderso left a comment

Choose a reason for hiding this comment

Uh oh!

jtmcdole commented Jan 8, 2026

Uh oh!

jtmcdole commented Jan 8, 2026

Uh oh!

jtmcdole commented Jan 9, 2026

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

10 participants

vhaudiquet commented Nov 18, 2025 •

edited

Loading

vhaudiquet commented Nov 19, 2025 •

edited

Loading

vhaudiquet commented Nov 25, 2025 •

edited

Loading

vhaudiquet commented Nov 25, 2025 •

edited

Loading

loic-sharma commented Nov 25, 2025 •

edited

Loading