Skip to content

Conversation

@sbaranov
Copy link
Contributor

@sbaranov sbaranov commented Oct 5, 2018

This fixes #19818 by ensuring that VM snapshot is copied from artifacts before gradle merges assets.

… assets.

This deflakes old behavior that suffered from nondeterminism because of reliance on the order of siblings in the dependency graph.

As a side effect, this also undoes the hack introduced in #14085 that increased build time by churning the output tree. This hack is no longer necessary since --preview-dart-2 flag is gone.
@sbaranov sbaranov changed the title Ensure that VM snapshot is copied from artifacts before gradle merges assets. Gradle plugin support for adding flutter as subproject to another Android app Oct 9, 2018
@sbaranov sbaranov requested a review from jason-simmons October 9, 2018 17:28
@sbaranov sbaranov merged commit 9a22f5a into flutter:master Oct 9, 2018
@sbaranov sbaranov deleted the fabric-io-compat branch October 9, 2018 17:43
@sbaranov
Copy link
Contributor Author

sbaranov commented Oct 9, 2018

This should also fix #18101.

liyuqian added a commit that referenced this pull request Oct 9, 2018
@passsy
Copy link
Contributor

passsy commented Oct 31, 2018

@liyuqian What's the reason for the revert? I'm running again in #19818 on current master

@liyuqian
Copy link
Contributor

@passsy I didn't revert the change here. It's probably a Github revert message bug in #22872

@liyuqian
Copy link
Contributor

I confirmed that the relevant change is still in the tip-of-tree flutter/flutter.

@filipproch
Copy link

I can also confirm that I have the current master and still running into the issue #19818

@sbaranov
Copy link
Contributor Author

sbaranov commented Nov 1, 2018

@passsy @filipproch Are you sure you're seeing the same exact issue? Are you're following the same exact steps outlined in #19818, adding fabric.io, etc?

@filipproch
Copy link

@sbaranov the error message is the same, the difference is whether I have apply fabric plugin or not. If I comment out the apply, the app launches. I can try to collect more info.

@sbaranov
Copy link
Contributor Author

sbaranov commented Nov 1, 2018

Yeah, we'd need a small example that reproduces the problem for us since we don't know what other things might still be affecting this. For example, are you seeing this problem if you follow the Add-to-App tutorial from scratch, and then only add fabric.io plugin?

@manondidi
Copy link

#24268

@aki-chang-dev
Copy link

Still got #19818 problems. And flutter-doctor shows everything is fine. The problem of my project is missing of flutter_assets. What else can I provide to help this issue?

@zoechi zoechi added tool Affects the "flutter" command-line tool. See also t: labels. t: gradle "flutter build" and "flutter run" on Android labels Jan 18, 2019
@jason-simmons
Copy link
Member

See #18025

Try upgrading to the current master branch including this commit:
f4c5e3e

@aki-chang-dev
Copy link

@jason-simmons I have the latest master branch and the issue still exists. When I create a whole new Android project as host app, it works, But my old project not. I explored the apk build by my old project, and flutter_assets folder is not included. I cannot figure out why for now.

@abouda
Copy link

abouda commented Feb 14, 2019

Issue still exists in latest master (Flutter (Channel master, v1.2.1-pre.179)

@github-actions github-actions bot locked as resolved and limited conversation to collaborators Aug 8, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

t: gradle "flutter build" and "flutter run" on Android tool Affects the "flutter" command-line tool. See also t: labels.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Crash App when add flutter module to existing apps

10 participants