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

Manual roll of src/third_party/dart from ed1e0e199e to b28b9ae8fc1#13171

Closed
mkustermann wants to merge 1 commit into
flutter-team-archive:masterfrom
mkustermann:roll-d26
Closed

Manual roll of src/third_party/dart from ed1e0e199e to b28b9ae8fc1#13171
mkustermann wants to merge 1 commit into
flutter-team-archive:masterfrom
mkustermann:roll-d26

Conversation

@mkustermann

@mkustermann mkustermann commented Oct 16, 2019

Copy link
Copy Markdown
Contributor
dart-lang/sdk@b28b9ae8fc Fix analyzer on windows
dart-lang/sdk@504c45fe2b [CFE] Don't cleanup moved parts in uriToSource
dart-lang/sdk@f3294f7083 [cfe] Ignore context when de-duplicating error messages
dart-lang/sdk@ed1e0e199e [vm/ffi] Handle IntConverter's similar to LoadUntagged in load optimizer

@dcharkes dcharkes left a comment

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.

dart_ffi_tag is not updated because it has been rolled already earlier today.

dart-lang/sdk@b28b9ae8fc Fix analyzer on windows
dart-lang/sdk@504c45fe2b [CFE] Don't cleanup moved parts in uriToSource
dart-lang/sdk@f3294f7083 [cfe] Ignore context when de-duplicating error messages
dart-lang/sdk@ed1e0e199e [vm/ffi] Handle IntConverter's similar to LoadUntagged in load optimizer
@mkustermann mkustermann requested a review from bkonyi October 16, 2019 17:32
@mkustermann mkustermann changed the title Manual roll of src/third_party/dart from 41b65b27c20 to b28b9ae8fc1 Manual roll of src/third_party/dart from ed1e0e199e to b28b9ae8fc1 Oct 16, 2019

@bkonyi bkonyi left a comment

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.

LGTM, but is there any particular reason for this manual roll? The autoroller should be able to pick this up in ~1 hour.

@bkonyi

bkonyi commented Oct 16, 2019

Copy link
Copy Markdown
Contributor

The autoroller is picking up this commit range in PR #13178.

@bkonyi bkonyi closed this Oct 16, 2019
@mkustermann

Copy link
Copy Markdown
Contributor Author

LGTM, but is there any particular reason for this manual roll?
The autoroller should be able to pick this up in ~1 hour.

The autoroller seemed to not have rolled in the last 24h and we wanted to get this in to unbreak golem. Good to know that it's rolling again.

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