Skip to content

Revert "[flutter_tools] cache flutter sdk version to disk" - #129130

Closed
whesse wants to merge 1 commit into
masterfrom
revert-124558-add-version-json
Closed

whesse wants to merge 1 commit into
masterfrom
revert-124558-add-version-json

Conversation

@whesse

@whesse whesse commented Jun 19, 2023

Copy link
Copy Markdown
Contributor

Reverts #124558

This causes a reproducible failure of packages/flutter_tools/test/commands.shard/permeable/upgrade_test.dart.
The failure of

'UpgradeCommandRunner fetchLatestVersion returns version if git succeeds'

with a failing null check in

#0 UpgradeCommandRunner.fetchLatestVersion (package:flutter_tools/src/commands/upgrade.dart:301:36)

disappears if the commit is reverted.

@github-actions github-actions Bot added a: text input Entering text in a text field or keyboard related problems tool Affects the "flutter" command-line tool. See also t: labels. labels Jun 19, 2023
@CaseyHillers CaseyHillers added warning: land on red to fix tree breakage autosubmit Merge PR when tree becomes green via auto submit App labels Jun 19, 2023

@jason-simmons jason-simmons left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

This test suite was failing because of a new random ordering seed.

See #129131

@jason-simmons jason-simmons removed the autosubmit Merge PR when tree becomes green via auto submit App label Jun 19, 2023
@whesse

whesse commented Jun 20, 2023

Copy link
Copy Markdown
Contributor Author

This failure was fixed forward in #129131. The revert is not needed

@whesse whesse closed this Jun 20, 2023
@guidezpl
guidezpl deleted the revert-124558-add-version-json branch April 18, 2024 22:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

a: text input Entering text in a text field or keyboard related problems tool Affects the "flutter" command-line tool. See also t: labels.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants