Resolve resolve native Flutter dependencies in Android Studio #167332

rekire · 2025-04-17T07:54:23Z

With this fix you can edit Flutter and native code in one Android Studio instance. This fixes #19830

The only downside is that you need to copy the files settings.gradle, build.gradle and local.properteis from the android folder to the project root folder. The last one should be added to the .gitignore file.

The second step is that you need to correct the path in the settings.gradle at the end is:

include ':app'

That needs to be replaced with:

include ':app'
project(':app').projectDir = new File(rootDir, 'android/app/')

The cool thing is if you have two flutter projects or another gradle module you can include them there too and you get the autocompletion too.

include ':app', ':app2', ':native-lib'
project(':app').projectDir = new File(rootDir, 'android/app/')
project(':app2').projectDir = new File(rootDir, 'app2/android/app/')
project(':native-lib').projectDir = new File(rootDir, 'example/native-lib/')

My assumption is that there will be some change requests and some questions. Feel free to ask

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.

…modules

flutter-dashboard · 2025-04-17T07:54:26Z

It looks like this pull request may not have tests. Please make sure to add tests or get an explicit test exemption before merging.

If you are not sure if you need tests, consider this rule of thumb: the purpose of a test is to make sure someone doesn't accidentally revert the fix. Ask yourself, is there anything in your PR that you feel it is important we not accidentally revert back to how it was before your fix?

Reviewers: Read the Tree Hygiene page and make sure this patch meets those guidelines before LGTMing.If you believe this PR qualifies for a test exemption, contact "@test-exemption-reviewer" in the #hackers channel in Discord (don't just cc them here, they won't see it!). The test exemption team is a small volunteer group, so all reviewers should feel empowered to ask for tests, without delegating that responsibility entirely to the test exemption group.

reidbaker · 2025-04-25T15:52:36Z

It looks like this change caused test failures.

execution failed for task ':app:preReleaseBuild'.
            > The following Android dependencies are set to compileOnly which is not supported:
              -> io.flutter:arm64_v8a_debug:1.0.0-eee1cffceea3b53287c8097679a9c868c4831330
              -> io.flutter:armeabi_v7a_debug:1.0.0-eee1cffceea3b53287c8097679a9c868c4831330
              -> io.flutter:flutter_embedding_debug:1.0.0-eee1cffceea3b53287c8097679a9c868c4831330
              -> io.flutter:x86_64_debug:1.0.0-eee1cffceea3b53287c8097679a9c868c4831330
              -> io.flutter:x86_debug:1.0.0-eee1cffceea3b53287c8097679a9c868c4831330

rekire · 2025-04-28T08:31:40Z

I am not sure, how I can fix the test yet.

reidbaker

Tests need to pass.
Blocking this so I can review it when it is ready.

flutter-dashboard · 2025-04-28T14:34:45Z

This pull request has been changed to a draft. The currently pending flutter-gold status will not be able to resolve until a new commit is pushed or the change is marked ready for review again.

For more guidance, visit Writing a golden file test for package:flutter.

Reviewers: Read the Tree Hygiene page and make sure this patch meets those guidelines before LGTMing.

rekire · 2025-05-03T12:32:38Z

@reidbaker I am not sure why some tests are red it changes on each master merge. Are the tests currently flaky or is it somehow my fault?

rekire · 2025-05-04T19:56:29Z

JFYI: I'm also working on migrating the default template to put the files at the right place. This also includes some fixes. I am not sure if I should make multiple PRs for that since I guess that will fix most likely two other bugs: rekire#1

Like that the .cxx is not on the ignore list or that you get the nasty "Missing ExternalProject for :" error.

That diff will also give you an idea what is required to make use of my change.

reidbaker · 2025-05-05T13:59:16Z

The tests I saw when I made #167332 (comment) looked like it was caused by your change. That said if a test fails then passes and your code unrelated then consider it flakey.

rekire · 2025-05-05T14:37:43Z

I merge the master one time more. That test you mentioned failed because of my change. I fixed my code so that test is green. Let's wait for the CI if I get all tests passed. (Actually I would prefer just to rerun the failed test to verify my flakiness assumption, but I don't have the permission)

What do you think should I change the project creation template within this PR?

What is the right way to make sure that the result of the current template won't break, when my changes become the default? My aimed "big" change is that the settings.gradle.kts and the local.properties file should move from the android to the project root directory.

reidbaker · 2025-05-05T14:48:52Z

packages/flutter_tools/gradle/src/main/kotlin/FlutterPlugin.kt

                    }
                }
            }
+        } catch (e: TaskInstantiationException) {
+            // ignored


Why is this safe to ignore? We use lockfiles in our own ci and having the generation fail without creating lockfiles would break our process.

That just happens when the task is registered twice which, can just happen when you have two flutter modules in your project (like a plugin). I tried to check if that task is already registered, but that caused some strange class cast exceptions in gradle, so I choose (without knowing a better solution) to ignore that special exception.

OK that seems safe to ignore. If I have time I can see if we can check if we ahve already applied the plugin which would catch the issue before the exception but you dont need to do that.

reidbaker · 2025-05-05T14:50:54Z

packages/flutter_tools/gradle/src/main/kotlin/FlutterPlugin.kt

@@ -97,6 +98,13 @@ class FlutterPlugin : Plugin<Project> {
            repositories.maven {
                url = uri(repository!!)
            }
+            if (plugins.hasPlugin("com.android.application") && hasProperty("android.injected.invoked.from.ide")) {


Where did you find "android.injected.invoked.from.ide"? I would like to read the source/documentation.

In general we dont do ide specific code in our plugin but from the string I cant tell if this is part of the api or something that is android studio specific.

I found it on StackOverflow: https://stackoverflow.com/q/18824890 I just want to inject the dependencies for Android Studio in normal builds like the CLI or CI those dependencies are not relevant.

@jwren Is there something equivalent for visual studio or something ide neutral you want this pr to include?
For now it is ok to leave this because I want @gmackall's opinion but I think we may ask that the check for android studio be removed to align with our philosophy of of shipping tools that are portable.

It also helps protect against bugs that only happen for users when they run from android studio. This is extra important because we do not have integration tests that require launch android studio and verifying the tooling works.

Source appears to be here https://cs.android.com/android-studio/platform/tools/base/+/mirror-goog-studio-main:build-system/builder-model/src/main/java/com/android/builder/model/InjectedProperties.kt?q=%20PROPERTY_INVOKED_FROM_IDE

If this change has been tested and seems to work then I am okay with it. Any other IDE neutral changes can be added as a follow on.

@rekire -- Thanks for this change. Did you check that this doesn't have any negative effects for Ultimate or Community?

Not yet, I can try it tomorrow on the CE, but I've no license for the ultimate edition.

CE is sufficient

I tested it with my new template and it worked as expected. I just had to click once on import gradle project or something similar which popped up on the right. I guess that is the default behavior. Visual Studio code seems not to break ether, however there I had no Kotlin support (most likely because I never installed those plugins)

flutter-dashboard · 2025-05-05T15:19:46Z

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.

For more guidance, visit Writing a golden file test for package:flutter.

Reviewers: Read the Tree Hygiene page and make sure this patch meets those guidelines before LGTMing.

Changes reported for pull request #167332 at sha 645a016

…io (flutter/flutter#167332)

Related to [Resolve resolve native Flutter dependencies in Android Studio](flutter/flutter#167332) and in preparation of [Extend template to resolve Dart and Kotlin](flutter/flutter#168490) to explain how to use that fix to resolve [Can't resolve symbol io.flutter.plugin dependency](flutter/flutter#19830) ## Presubmit checklist - [ ] If this PR is not meant to land until a future stable release, mark it as draft with an explanation. - [x] This PR doesn't contain automatically generated corrections (generated by Grammarly or similar). - [x] This PR follows the [Google Developer Documentation Style Guidelines](https://developers.google.com/style)—for example, it doesn't use _i.e._ or _e.g._, and it avoids _I_ and _we_ (first-person pronouns). - [x] This PR uses [semantic line breaks](https://github.com/dart-lang/site-shared/blob/main/doc/writing-for-dart-and-flutter-websites.md#semantic-line-breaks) of 80 characters or fewer. --------- Co-authored-by: Shams Zakhour (ignore Sfshaza) <44418985+sfshaza2@users.noreply.github.com>

… support (#172651) I asked for this in this comment #167332 (comment) but the request may have been unclear, as the PR landed without the change. Following up to structure based on my comment. I want to avoid the configuration that is specific to this one use case creeping out across the FGP and wider flutter tool if it doesn't need to. ## Pre-launch Checklist - [x] I read the [Contributor Guide] and followed the process outlined there for submitting PRs. - [x] I read the [Tree Hygiene] wiki page, which explains my responsibilities. - [x] I read and followed the [Flutter Style Guide], including [Features we expect every widget to implement]. - [x] I signed the [CLA]. - [x] I listed at least one issue that this PR fixes in the description above. - [x] 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]. - [x] I followed the [breaking change policy] and added [Data Driven Fixes] where supported. - [x] All existing and new tests are passing. If you need help, consider asking for advice on the #hackers-new channel on [Discord].  [Contributor Guide]: https://github.com/flutter/flutter/blob/main/docs/contributing/Tree-hygiene.md#overview [Tree Hygiene]: https://github.com/flutter/flutter/blob/main/docs/contributing/Tree-hygiene.md [test-exempt]: https://github.com/flutter/flutter/blob/main/docs/contributing/Tree-hygiene.md#tests [Flutter Style Guide]: https://github.com/flutter/flutter/blob/main/docs/contributing/Style-guide-for-Flutter-repo.md [Features we expect every widget to implement]: https://github.com/flutter/flutter/blob/main/docs/contributing/Style-guide-for-Flutter-repo.md#features-we-expect-every-widget-to-implement [CLA]: https://cla.developers.google.com/ [flutter/tests]: https://github.com/flutter/tests [breaking change policy]: https://github.com/flutter/flutter/blob/main/docs/contributing/Tree-hygiene.md#handling-breaking-changes [Discord]: https://github.com/flutter/flutter/blob/main/docs/contributing/Chat.md [Data Driven Fixes]: https://github.com/flutter/flutter/blob/main/docs/contributing/Data-driven-Fixes.md Co-authored-by: Gray Mackall <mackall@google.com>

… support (flutter#172651) I asked for this in this comment flutter#167332 (comment) but the request may have been unclear, as the PR landed without the change. Following up to structure based on my comment. I want to avoid the configuration that is specific to this one use case creeping out across the FGP and wider flutter tool if it doesn't need to. ## Pre-launch Checklist - [x] I read the [Contributor Guide] and followed the process outlined there for submitting PRs. - [x] I read the [Tree Hygiene] wiki page, which explains my responsibilities. - [x] I read and followed the [Flutter Style Guide], including [Features we expect every widget to implement]. - [x] I signed the [CLA]. - [x] I listed at least one issue that this PR fixes in the description above. - [x] 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]. - [x] I followed the [breaking change policy] and added [Data Driven Fixes] where supported. - [x] All existing and new tests are passing. If you need help, consider asking for advice on the #hackers-new channel on [Discord].  [Contributor Guide]: https://github.com/flutter/flutter/blob/main/docs/contributing/Tree-hygiene.md#overview [Tree Hygiene]: https://github.com/flutter/flutter/blob/main/docs/contributing/Tree-hygiene.md [test-exempt]: https://github.com/flutter/flutter/blob/main/docs/contributing/Tree-hygiene.md#tests [Flutter Style Guide]: https://github.com/flutter/flutter/blob/main/docs/contributing/Style-guide-for-Flutter-repo.md [Features we expect every widget to implement]: https://github.com/flutter/flutter/blob/main/docs/contributing/Style-guide-for-Flutter-repo.md#features-we-expect-every-widget-to-implement [CLA]: https://cla.developers.google.com/ [flutter/tests]: https://github.com/flutter/tests [breaking change policy]: https://github.com/flutter/flutter/blob/main/docs/contributing/Tree-hygiene.md#handling-breaking-changes [Discord]: https://github.com/flutter/flutter/blob/main/docs/contributing/Chat.md [Data Driven Fixes]: https://github.com/flutter/flutter/blob/main/docs/contributing/Data-driven-Fixes.md Co-authored-by: Gray Mackall <mackall@google.com>

…io (flutter/flutter#167332)

… support (flutter#172651) I asked for this in this comment flutter#167332 (comment) but the request may have been unclear, as the PR landed without the change. Following up to structure based on my comment. I want to avoid the configuration that is specific to this one use case creeping out across the FGP and wider flutter tool if it doesn't need to. ## Pre-launch Checklist - [x] I read the [Contributor Guide] and followed the process outlined there for submitting PRs. - [x] I read the [Tree Hygiene] wiki page, which explains my responsibilities. - [x] I read and followed the [Flutter Style Guide], including [Features we expect every widget to implement]. - [x] I signed the [CLA]. - [x] I listed at least one issue that this PR fixes in the description above. - [x] 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]. - [x] I followed the [breaking change policy] and added [Data Driven Fixes] where supported. - [x] All existing and new tests are passing. If you need help, consider asking for advice on the #hackers-new channel on [Discord].  [Contributor Guide]: https://github.com/flutter/flutter/blob/main/docs/contributing/Tree-hygiene.md#overview [Tree Hygiene]: https://github.com/flutter/flutter/blob/main/docs/contributing/Tree-hygiene.md [test-exempt]: https://github.com/flutter/flutter/blob/main/docs/contributing/Tree-hygiene.md#tests [Flutter Style Guide]: https://github.com/flutter/flutter/blob/main/docs/contributing/Style-guide-for-Flutter-repo.md [Features we expect every widget to implement]: https://github.com/flutter/flutter/blob/main/docs/contributing/Style-guide-for-Flutter-repo.md#features-we-expect-every-widget-to-implement [CLA]: https://cla.developers.google.com/ [flutter/tests]: https://github.com/flutter/tests [breaking change policy]: https://github.com/flutter/flutter/blob/main/docs/contributing/Tree-hygiene.md#handling-breaking-changes [Discord]: https://github.com/flutter/flutter/blob/main/docs/contributing/Chat.md [Data Driven Fixes]: https://github.com/flutter/flutter/blob/main/docs/contributing/Data-driven-Fixes.md Co-authored-by: Gray Mackall <mackall@google.com>

rekire added 2 commits April 17, 2025 09:39

Add compileOnly dependencies to resolve the flutter engine imports

1883cf1

Check if generateLockfiles exists for projects with multiple flutter …

3874f13

…modules

github-actions bot added platform-android Android applications specifically tool Affects the "flutter" command-line tool. See also t: labels. labels Apr 17, 2025

camsim99 requested review from reidbaker and gmackall and removed request for reidbaker April 22, 2025 18:14

reidbaker requested changes Apr 28, 2025

View reviewed changes

rekire marked this pull request as draft April 28, 2025 14:34

rekire added 6 commits April 29, 2025 21:46

Only add compileOnly dependencies when Android Studio is detected

84090fc

Merge branch 'master' into resolve-resolve-flutter-dependencies

73a5826

Merge branch 'master' into resolve-resolve-flutter-dependencies

65be9f2

Merge branch 'master' into resolve-resolve-flutter-dependencies

6dd2df4

Catch the exception when the task is already registered

b8b4590

Merge branch 'master' into resolve-resolve-flutter-dependencies

0208307

rekire marked this pull request as ready for review May 3, 2025 12:29

Merge branch 'master' into resolve-resolve-flutter-dependencies

645a016

reidbaker reviewed May 5, 2025

View reviewed changes

reidbaker requested changes May 5, 2025

View reviewed changes

flutter-dashboard bot added the will affect goldens Changes to golden files label May 5, 2025

engine-flutter-autoroll mentioned this pull request Jun 5, 2025

Roll Flutter from 0e536eb9fe4b to 6e738ee3929f (144 revisions) flutter/packages#9381

Closed

engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request Jun 5, 2025

0c0406a61 Resolve resolve native Flutter dependencies in Android Stud…

0e39e8b

…io (flutter/flutter#167332)

engine-flutter-autoroll mentioned this pull request Jun 5, 2025

Roll Flutter from 0e536eb9fe4b to 121caa68d6c1 (145 revisions) flutter/packages#9382

Closed

engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request Jun 5, 2025

0c0406a61 Resolve resolve native Flutter dependencies in Android Stud…

498e58b

…io (flutter/flutter#167332)

engine-flutter-autoroll mentioned this pull request Jun 5, 2025

Roll Flutter from 0e536eb9fe4b to 736053538b14 (146 revisions) flutter/packages#9383

Closed

engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request Jun 5, 2025

0c0406a61 Resolve resolve native Flutter dependencies in Android Stud…

6122665

…io (flutter/flutter#167332)

engine-flutter-autoroll mentioned this pull request Jun 5, 2025

Roll Flutter from 0e536eb9fe4b to 817fa9701467 (147 revisions) flutter/packages#9384

Closed

engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request Jun 5, 2025

0c0406a61 Resolve resolve native Flutter dependencies in Android Stud…

2942c4c

…io (flutter/flutter#167332)

engine-flutter-autoroll mentioned this pull request Jun 5, 2025

Roll Flutter from 0e536eb9fe4b to 92c55f06e01e (148 revisions) flutter/packages#9385

Closed

engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request Jun 5, 2025

0c0406a61 Resolve resolve native Flutter dependencies in Android Stud…

d61d9fb

…io (flutter/flutter#167332)

engine-flutter-autoroll mentioned this pull request Jun 5, 2025

Roll Flutter from 0e536eb9fe4b to 4469c5e4db3e (149 revisions) flutter/packages#9386

Closed

engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request Jun 5, 2025

0c0406a61 Resolve resolve native Flutter dependencies in Android Stud…

4f03ac4

…io (flutter/flutter#167332)

engine-flutter-autoroll mentioned this pull request Jun 5, 2025

Roll Flutter from 0e536eb9fe4b to 63980cb25ba5 (150 revisions) flutter/packages#9388

Closed

engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request Jun 5, 2025

0c0406a61 Resolve resolve native Flutter dependencies in Android Stud…

c6d7f01

…io (flutter/flutter#167332)

engine-flutter-autoroll mentioned this pull request Jun 5, 2025

Roll Flutter from 0e536eb9fe4b to 7f26571eb13a (151 revisions) flutter/packages#9390

Closed

engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request Jun 5, 2025

0c0406a61 Resolve resolve native Flutter dependencies in Android Stud…

5c778bb

…io (flutter/flutter#167332)

Piinks mentioned this pull request Jun 6, 2025

Flutter Notable Commits - 2025 #161460

Open

This was referenced Jul 14, 2025

Fix handling of generate lock files #172101

Closed

Add migration guide for moving the gradle files flutter/website#12203

Merged

rekire mentioned this pull request Jul 14, 2025

Fix handling of generate lock files #172124

Draft

9 tasks

gmackall mentioned this pull request Jul 23, 2025

[FGP] Small restructuring of Android Studio native flutter dependency support #172651

Merged

9 tasks

engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request Aug 14, 2025

0c0406a61 Resolve resolve native Flutter dependencies in Android Stud…

660bc99

…io (flutter/flutter#167332)

engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request Aug 14, 2025

0c0406a61 Resolve resolve native Flutter dependencies in Android Stud…

4d54484

…io (flutter/flutter#167332)

engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request Aug 15, 2025

0c0406a61 Resolve resolve native Flutter dependencies in Android Stud…

1429d8e

…io (flutter/flutter#167332)

engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request Aug 15, 2025

0c0406a61 Resolve resolve native Flutter dependencies in Android Stud…

f816202

…io (flutter/flutter#167332)

engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request Aug 16, 2025

0c0406a61 Resolve resolve native Flutter dependencies in Android Stud…

01bed8f

…io (flutter/flutter#167332)

Resolve resolve native Flutter dependencies in Android Studio #167332

Resolve resolve native Flutter dependencies in Android Studio #167332

Conversation

rekire commented Apr 17, 2025 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Pre-launch Checklist

Uh oh!

flutter-dashboard bot commented Apr 17, 2025

Uh oh!

reidbaker commented Apr 25, 2025

Uh oh!

rekire commented Apr 28, 2025

Uh oh!

reidbaker left a comment

Choose a reason for hiding this comment

Uh oh!

flutter-dashboard bot commented Apr 28, 2025

Uh oh!

rekire commented May 3, 2025

Uh oh!

rekire commented May 4, 2025 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

reidbaker commented May 5, 2025

Uh oh!

rekire commented May 5, 2025

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

flutter-dashboard bot commented May 5, 2025

Uh oh!

Uh oh!

rekire commented Apr 17, 2025 •

edited

Loading

rekire commented May 4, 2025 •

edited

Loading