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

Copy //dart-pkg/zircon|fuchsia from Topaz into the engine. - #8884

Merged
chinmaygarde merged 2 commits into
flutter-team-archive:masterfrom
chinmaygarde:fdk_land_runtime_dart_pkg
May 7, 2019
Merged

chinmaygarde merged 2 commits into
flutter-team-archive:masterfrom
chinmaygarde:fdk_land_runtime_dart_pkg

Conversation

@chinmaygarde

Copy link
Copy Markdown
Contributor

Build rules and include paths have been updated as necessary. No other functional change.

native 'System_ChannelCreate';
static HandleResult channelFromFile(String path)
native 'System_ChannelFromFile';
static int reboot()

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.

Are you planning to include updates still happening in the Fuchsia tree like https://fuchsia-review.googlesource.com/c/topaz/+/275857 ?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Yes. I am keeping a list of these patches in the "Patches to Backport" section of go/fdk-notes.


zircon_sdk_ext_files = [
zircon_sdk_ext_lib,
"//topaz/public/dart-pkg/zircon/lib/src/handle.dart",

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.

are these right?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

I dont think this file is being referenced. I'll get rid of it.

@zanderso

zanderso commented May 7, 2019

Copy link
Copy Markdown
Contributor

Oh, also, are the years in the copyright headers correct?

@chinmaygarde

Copy link
Copy Markdown
Contributor Author

Oh, also, are the years in the copyright headers correct?

Yes. Verified with @Hixie

Build rules and include paths have been updated as necessary. No other functional change.
@chinmaygarde
chinmaygarde force-pushed the fdk_land_runtime_dart_pkg branch from f2fd003 to 66a74a2 Compare May 7, 2019 21:10
@chinmaygarde
chinmaygarde merged commit 0c7a3f2 into flutter-team-archive:master May 7, 2019
@chinmaygarde
chinmaygarde deleted the fdk_land_runtime_dart_pkg branch May 7, 2019 22:06
engine-flutter-autoroll added a commit to flutter/flutter that referenced this pull request May 8, 2019
flutter-team-archive/engine@149b7b8...0fc6efb

git log 149b7b8..0fc6efb --no-merges --oneline
0fc6efb Roll src/third_party/dart a8b6e18b62..8143450941 (11 commits)
93d1c37 Remove absolute path in new Fuchsia SDK based runner target dependency. (flutter-team-archive/engine#8888)
0b406c3 Copy the Flutter Runner from //topaz into the engine. (flutter-team-archive/engine#8886)
2b1f992 new lints (flutter-team-archive/engine#8849)
0c7a3f2 Copy //dart-pkg/zircon|fuchsia from Topaz into the engine. (flutter-team-archive/engine#8884)
ba82ea2 Roll src/third_party/skia 056944747072..51e15a69ce5b (8 commits) (flutter-team-archive/engine#8885)
d717044 Copy //runtime/dart/utils from Topaz into the engine. (flutter-team-archive/engine#8871)

The AutoRoll server is located here: https://autoroll.skia.org/r/flutter-engine-flutter-autoroll

Documentation for the AutoRoller is here:
https://skia.googlesource.com/buildbot/+/master/autoroll/README.md

If the roll is causing failures, please contact the current sheriff ([email protected]), and stop
the roller if necessary.
huqiuser pushed a commit to huqiuser/engine that referenced this pull request Jun 12, 2019
…eam-archive#8884)

Build rules and include paths have been updated as necessary. No other functional change.
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