Skip to content

[flutter_tools] Reland migrate xcode_backend.sh to flutter assemble#50200

Merged
jonahwilliams merged 8 commits intoflutter:masterfrom
jonahwilliams:full_assemble
Feb 5, 2020
Merged

[flutter_tools] Reland migrate xcode_backend.sh to flutter assemble#50200
jonahwilliams merged 8 commits intoflutter:masterfrom
jonahwilliams:full_assemble

Conversation

@jonahwilliams
Copy link
Copy Markdown
Contributor

Description

Updates xcode_backend.sh to use (mostly) assemble APIs. This almost completes the migration, though I still need to move the asset unpack step - this is slightly more complicated due to the module format.

Should have no new external behavior, but does allow us to more easily wire up new features like font-subset, dart-defines, and obfuscation (which was never implemented for iOS?)

FYI @dnfield

Fixes #32925

Reland: Makes the Universal framework depend on kernel. Due to the behavior of the codegen integration, running them at the same time causes parts of .dart_tool to be deleted and recreated.

@fluttergithubbot fluttergithubbot added the tool Affects the "flutter" command-line tool. See also t: labels. label Feb 5, 2020
Copy link
Copy Markdown
Contributor

@dnfield dnfield left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

Copy link
Copy Markdown
Member

@jmagman jmagman left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Checked out this branch. Running in debug and release on a real device works, archiving works. LGTM

@jonahwilliams jonahwilliams merged commit 868b0e6 into flutter:master Feb 5, 2020
@jonahwilliams jonahwilliams deleted the full_assemble branch February 5, 2020 21:04
jonahwilliams pushed a commit that referenced this pull request Feb 5, 2020
jonahwilliams pushed a commit that referenced this pull request Feb 5, 2020
@jonahwilliams jonahwilliams restored the full_assemble branch February 6, 2020 00:08
@github-actions github-actions Bot locked as resolved and limited conversation to collaborators Aug 1, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

tool Affects the "flutter" command-line tool. See also t: labels.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Re-implement existing iOS and Android build in terms of --legacy flutter assemble targets

5 participants