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

Copy //runtime/dart/utils from Topaz into the engine. - #8871

Merged
chinmaygarde merged 5 commits into
flutter-team-archive:masterfrom
chinmaygarde:fdk_land_runtime_dart_utils
May 7, 2019
Merged

chinmaygarde merged 5 commits into
flutter-team-archive:masterfrom
chinmaygarde:fdk_land_runtime_dart_utils

Conversation

@chinmaygarde

Copy link
Copy Markdown
Contributor

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

Build rules and include paths have been updated as necessary. No other functional change.
// Use of this source code is governed by a BSD-style license that can be
// found in the LICENSE file.

#ifndef TOPAZ_RUNTIME_DART_UTILS_FILES_H_

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.

Should the include guards be updated to not mention Topaz?

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.

Updated.

@chinmaygarde

Copy link
Copy Markdown
Contributor Author

Header guards have been updated. PTAL.

@chinmaygarde

Copy link
Copy Markdown
Contributor Author

Presubmits have been failing because of 502 on DEPS.

@chinmaygarde

Copy link
Copy Markdown
Contributor Author

I have addressed all issues, lgtm?

@chinmaygarde
chinmaygarde merged commit d717044 into flutter-team-archive:master May 7, 2019
@chinmaygarde
chinmaygarde deleted the fdk_land_runtime_dart_utils branch May 7, 2019 20:59
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
…rchive#8871)

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