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

Conversation

alexmarkov
Copy link
Contributor

Flutter downloads only one version of flutter_patched_sdk, which was obtained from host_debug build on Linux. Generated entry points JSON files are different in profile and release builds (as release build omits observatory), so these files should be downloaded independently of flutter_patched_sdk.

This PR moves generated entry points JSON files from flutter_patched_sdk to a separate directory (out/*/dart_entry_points) and adds an option to specify entry points to frontend_server (as Flutter tools will need to pass custom locations of entry points files outside of flutter_patched_sdk).

…ption to pass entry points to frontend_server
@alexmarkov alexmarkov requested review from aam and zanderso March 1, 2018 21:51
Copy link
Member

@zanderso zanderso left a comment

Choose a reason for hiding this comment

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

lgtm but isn't aam moving this code?

@alexmarkov alexmarkov merged commit f7c5fea into flutter:master Mar 1, 2018
@alexmarkov alexmarkov deleted the entry-points-artifacts branch March 1, 2018 22:02
@aam
Copy link
Member

aam commented Mar 1, 2018

I have introduced this into https://dart-review.googlesource.com/c/sdk/+/43888

@alexmarkov
Copy link
Contributor Author

I have introduced this into https://dart-review.googlesource.com/c/sdk/+/43888

Thanks!

reed-at-google pushed a commit to reed-at-google/engine that referenced this pull request Mar 14, 2018
…ption to pass entry points to frontend_server (flutter#4734)

Flutter downloads only one version of flutter_patched_sdk, which was obtained from host_debug build on Linux. Generated entry points JSON files are different in profile and release builds (as release build omits observatory), so these files should be downloaded independently of flutter_patched_sdk.

This PR moves generated entry points JSON files from flutter_patched_sdk to a separate directory (out/*/dart_entry_points) and adds an option to specify entry points to frontend_server (as Flutter tools will need to pass custom locations of entry points files outside of flutter_patched_sdk).
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants