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

Reland "Use single_root_scheme when compiling platform (#6402)" #6473

Merged
merged 1 commit into from
Oct 9, 2018

Conversation

aam
Copy link
Member

@aam aam commented Oct 9, 2018

This reverts commit 5340f42 (#6431), relands single root filesystem that is needed for reusable platform kernel files, that is safe to land now since dart roll c048bec has the fix for previously broken dependency file generation.

…er#6402)" (flutter#6431)"

This reverts commit 5340f42, relands single root filesystem that is needed for reusable platform kernel files, that is safe to land now since dart roll c048bec has the fix for previously broken dependency file generation.
@aam aam requested a review from jensjoha October 9, 2018 05:23
Copy link
Contributor

@jensjoha jensjoha left a comment

Choose a reason for hiding this comment

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

I personally don't like the "revert revert" title --- but I don't know if there's a standard on whether that's the way to do it or not.

@aam aam changed the title Revert "Revert "Use single_root_scheme when compiling platform (#6402… Reland "Use single_root_scheme when compiling platform (#6402)" Oct 9, 2018
@aam
Copy link
Member Author

aam commented Oct 9, 2018

https://cirrus-ci.com/task/5462978816114688 seems to be green, yet Cirrus status on github is not being updated. Ignoring.

@aam aam merged commit 289c6b1 into flutter:master Oct 9, 2018
@aam aam deleted the reland-single-root-fs branch October 9, 2018 15:08
engine-flutter-autoroll added a commit to engine-flutter-autoroll/flutter that referenced this pull request Oct 9, 2018
flutter/engine@08af410...289c6b1

git log 08af410..289c6b1 --no-merges --oneline
289c6b1 Revert "Revert "Use single_root_scheme when compiling platform (flutter#6402)" (flutter#6431)" (flutter/engine#6473)

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, who should
be CC'd on the roll, and stop the roller if necessary.
liyuqian pushed a commit to flutter/flutter that referenced this pull request Oct 9, 2018
flutter/engine@08af410...289c6b1

git log 08af410..289c6b1 --no-merges --oneline
289c6b1 Revert "Revert "Use single_root_scheme when compiling platform (#6402)" (#6431)" (flutter/engine#6473)

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, who should
be CC'd on the roll, and stop the roller if necessary.
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.

3 participants