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

Revert "Build AOT and test targets, generate FARs when building Fuchsia"#12781

Merged
dnfield merged 1 commit into
masterfrom
revert-12761-fuchsia_far
Oct 4, 2019
Merged

Revert "Build AOT and test targets, generate FARs when building Fuchsia"#12781
dnfield merged 1 commit into
masterfrom
revert-12761-fuchsia_far

Conversation

@dnfield

@dnfield dnfield commented Oct 4, 2019

Copy link
Copy Markdown
Contributor

Reverts #12761

This broke CI. I believe because of the python changes - we may have to leave that script as-is, update LUCI, and then reland the changes to the script. For now, just will revert to unblock tree.

TBR @iskakaushik @chinmaygarde @gw280

@dnfield
dnfield merged commit 2e163b2 into master Oct 4, 2019
@dnfield
dnfield deleted the revert-12761-fuchsia_far branch October 4, 2019 00:20
engine-flutter-autoroll added a commit to engine-flutter-autoroll/flutter that referenced this pull request Oct 4, 2019
engine-flutter-autoroll added a commit to flutter/flutter that referenced this pull request Oct 4, 2019
[email protected]:flutter/engine.git/compare/62e58c5d9fc2...2e163b2

git log 62e58c5..2e163b2 --no-merges --oneline
2019-10-04 [email protected] Revert "Build AOT and test targets, generate FARs when building Fuchsia (#12761)" (flutter-team-archive/engine#12781)
2019-10-03 [email protected] Enabled people to chose if SystemNavigator.pop is animated on iOS. (flutter-team-archive/engine#12752)
2019-10-03 [email protected] Added some thread asserts to the code and made ios_surface_ safe since (flutter-team-archive/engine#12775)
2019-10-03 [email protected] Build AOT and test targets, generate FARs when building Fuchsia (flutter-team-archive/engine#12761)


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
dnfield added a commit to dnfield/engine that referenced this pull request Oct 4, 2019
Inconnu08 pushed a commit to Inconnu08/flutter that referenced this pull request Nov 26, 2019
[email protected]:flutter/engine.git/compare/62e58c5d9fc2...2e163b2

git log 62e58c5..2e163b2 --no-merges --oneline
2019-10-04 [email protected] Revert "Build AOT and test targets, generate FARs when building Fuchsia (flutter#12761)" (flutter-team-archive/engine#12781)
2019-10-03 [email protected] Enabled people to chose if SystemNavigator.pop is animated on iOS. (flutter-team-archive/engine#12752)
2019-10-03 [email protected] Added some thread asserts to the code and made ios_surface_ safe since (flutter-team-archive/engine#12775)
2019-10-03 [email protected] Build AOT and test targets, generate FARs when building Fuchsia (flutter-team-archive/engine#12761)


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
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.

2 participants