Skip to content
This repository was archived by the owner on Feb 25, 2025. It is now read-only.

Revert "Re-land "Support multiple shells in a single process. (#4932)""#4981

Merged
mraleph merged 1 commit into
masterfrom
revert-4977-master
Apr 12, 2018
Merged

Revert "Re-land "Support multiple shells in a single process. (#4932)""#4981
mraleph merged 1 commit into
masterfrom
revert-4977-master

Conversation

@mraleph

@mraleph mraleph commented Apr 12, 2018

Copy link
Copy Markdown
Contributor

Reverts #4977

This breaks hot restart.

To reproduce build the local engine with this change, run Flutter Gallery flutter run --debug --local-engine android_debug and once the application is loaded press R.

@mraleph

mraleph commented Apr 12, 2018

Copy link
Copy Markdown
Contributor Author

Originally discovered by @jensjoha

/cc @cbracken @a-siva

@mraleph mraleph merged commit 723c7d0 into master Apr 12, 2018
@mraleph mraleph deleted the revert-4977-master branch April 12, 2018 16:28
@jason-simmons

Copy link
Copy Markdown
Contributor

Hot restart is failing because the RunInView service protocol method is being passed a bogus .packages file along with the path to the recompiled main script .dill

@chinmaygarde @aam If RunInView is running a .dill, then should the .packages parameter be ignored? If so, then DartIsolate::PrepareForRunningFromSource can check for a .dill and then delegate to DartIsolate::PrepareForRunningFromSnapshot

@aam

aam commented Apr 12, 2018

Copy link
Copy Markdown
Contributor

@jason-simmons wrote:

If RunInView is running a .dill, then should the .packages parameter be ignored?

Yes, I believe so.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

Development

Successfully merging this pull request may close these issues.

4 participants