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

Ensure we export the various dart snapshot symbols on Fuchsia#15953

Merged
fluttergithubbot merged 1 commit into
flutter-team-archive:masterfrom
gw280:gwright-export-symbols
Jan 24, 2020
Merged

Ensure we export the various dart snapshot symbols on Fuchsia#15953
fluttergithubbot merged 1 commit into
flutter-team-archive:masterfrom
gw280:gwright-export-symbols

Conversation

@gw280

@gw280 gw280 commented Jan 24, 2020

Copy link
Copy Markdown
Contributor

When building in JIT mode, we package the dart isolate/vm snapshots in the binary and export the symbols for lookup using dlsym.

We need to ensure these symbols are exported correctly on Fuchsia so that shell_unittests can initalise properly.

@gw280
gw280 requested a review from iskakaushik January 24, 2020 00:03
@auto-assign
auto-assign Bot requested a review from liyuqian January 24, 2020 00:03

@iskakaushik iskakaushik left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

LGTM

@gw280 gw280 added the waiting for tree to go green This PR is approved and tested, but waiting for the tree to be green to land. label Jan 24, 2020
@fluttergithubbot
fluttergithubbot merged commit b4d2557 into flutter-team-archive:master Jan 24, 2020
engine-flutter-autoroll added a commit to flutter/flutter that referenced this pull request Jan 25, 2020
flutter-team-archive/engine@f10f03a...51a7964

git log f10f03a..51a7964 --first-parent --oneline
2020-01-24 [email protected] Roll src/third_party/dart bc9348829ef8..fc3af737c759 (2 commits) (flutter-team-archive/engine#15965)
2020-01-24 [email protected] Roll fuchsia/sdk/core/linux-amd64 from Wc7e4... to 8Ns10... (flutter-team-archive/engine#15964)
2020-01-24 [email protected] Roll src/third_party/skia a7e557f3e353..3ea4d5bb857d (4 commits) (flutter-team-archive/engine#15963)
2020-01-24 [email protected] Roll src/third_party/dart c359b5943a52..bc9348829ef8 (1 commits) (flutter-team-archive/engine#15962)
2020-01-24 [email protected] ensure we export the various dart snapshot symbols on Fuchsia (flutter-team-archive/engine#15953)
2020-01-24 [email protected] Roll src/third_party/dart 3eaae5405d37..c359b5943a52 (13 commits) (flutter-team-archive/engine#15959)
2020-01-24 [email protected] Migrate flutter_runner from flutter_runner::{Thread,Loop} to fml::{Thread,MessageLoop} (flutter-team-archive/engine#15118)
2020-01-24 [email protected] Re-arm timer as necessary in MessageLoopFuchsia
2020-01-24 [email protected] Roll src/third_party/skia c88a3bc3f561..a7e557f3e353 (6 commits) (flutter-team-archive/engine#15957)
2020-01-24 [email protected] Roll fuchsia/sdk/core/mac-amd64 from LAOYs... to 6_pZp... (flutter-team-archive/engine#15954)
2020-01-24 [email protected] Fixes FlutterCallbackInfomation leaks (flutter-team-archive/engine#15089)
2020-01-24 [email protected] Fixes oc leaks in platform plugin (flutter-team-archive/engine#15041)
2020-01-24 [email protected] Do not reset the child isolate preparer if the isolate group data already has one (flutter-team-archive/engine#15952)


If this roll has caused a breakage, revert this CL and stop the roller
using the controls here:
https://autoroll.skia.org/r/flutter-engine-flutter-autoroll
Please CC [email protected] on the revert to ensure that a human
is aware of the problem.

To report a problem with the AutoRoller itself, please file a bug:
https://bugs.chromium.org/p/skia/issues/entry?template=Autoroller+Bug

Documentation for the AutoRoller is here:
https://skia.googlesource.com/buildbot/+/master/autoroll/README.md
NoamDev added a commit to NoamDev/engine that referenced this pull request Feb 27, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

cla: yes waiting for tree to go green This PR is approved and tested, but waiting for the tree to be green to land.

Development

Successfully merging this pull request may close these issues.

4 participants