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

Conversation

@stuartmorgan-g
Copy link
Collaborator

Updates the Windows builds for the breaking change in
flutter/flutter#40194

Note: plugins continue to hard-code the cache dir instead of using
FLUTTER_EPHEMERAL_DIR since currently they share the generated
properties file, but need a project-local copy of the cache. That
whole structure needs to be reworked, so the fact that it's
currently hard-coded isn't worth trying to fix right now
(e.g., by making FLUTTER_EPHEMERAL_DIR relative).

Copy link

@jonahwilliams jonahwilliams left a comment

Choose a reason for hiding this comment

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

RSLGTM

@stuartmorgan-g stuartmorgan-g merged commit bf2fea9 into google:master Sep 11, 2019
@stuartmorgan-g stuartmorgan-g deleted the windows-ephemeral branch September 11, 2019 14:47
stuartmorgan-g added a commit to stuartmorgan-g/flutter-desktop-embedding that referenced this pull request Sep 12, 2019
This should have been done as part of google#553; it would have avoided google#556.

Since there have been several other recent desktop-related fixes, this
requires a new enough version to pick them up as well to reduce the
chance of people hitting them.
stuartmorgan-g added a commit that referenced this pull request Sep 12, 2019
This should have been done as part of #553; it would have avoided #556.

Since there have been several other recent desktop-related fixes, this
requires a new enough version to pick them up as well to reduce the
chance of people hitting them.
stuartmorgan-g added a commit to stuartmorgan-g/flutter-desktop-embedding that referenced this pull request Sep 12, 2019
Fixes wrapper source path updates missed in google#553.
stuartmorgan-g added a commit that referenced this pull request Sep 12, 2019
Fixes wrapper source path updates missed in #553.
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Aug 6, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants