Skip to content

Conversation

@mkustermann
Copy link
Member

This is a re-land with downgraded package:flutter_gallery_assets version, which appears to have a weired sdk constraint allowing this particular dart sdk to be compatible, namely it uses >=1.19.0 <2.1.0

This is a re-land with downgraded `package:flutter_gallery_assets`
version.
@mkustermann
Copy link
Member Author

There seems to be still issues, even though the presubmit checks don't cover it. I think it has to do with inconsistent usage of platform_strong.dill vs platform.dill, which might require me to first do another engine change before this can proceed.

description: A new flutter project.

environment:
# The pub client defaults to an <2.0.0 sdk constraint which we need to explicitly overwrite.
Copy link
Member

Choose a reason for hiding this comment

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

This is really strange we have a constraint like that. Could it be that it is 'older' pub clients that default to <2.0.0 sdk constraint, but we should not be relying on them?

Copy link
Member Author

Choose a reason for hiding this comment

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

This was added here:

"""
There is now a default SDK constraint of <2.0.0 for any package with no
existing upper bound. This allows us to move more safely to 2.0.0. All new
packages published on pub will now require an upper bound SDK constraint so
future major releases of Dart don't destabilize the package ecosystem.
"""

Copy link
Member

Choose a reason for hiding this comment

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

Ah, good to know. Thanks!

Copy link
Contributor

@tvolkert tvolkert left a comment

Choose a reason for hiding this comment

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

LGTM

@mkustermann mkustermann changed the title Reland "Roll engine to version b148e628ec86b3a9a0382e0bcfae73f0390a8232 (#20427)" Reland "Roll engine to version e3687f70c7ece72000b32ee1b3c02755ba5361ac (#20427)" Aug 15, 2018
@jonahwilliams jonahwilliams merged commit 9299c02 into flutter:master Aug 15, 2018
@Hixie
Copy link
Contributor

Hixie commented Aug 17, 2018

This regressed memory usage on the flutter_gallery__memory_nav benchmark quite severely.

@mkustermann
Copy link
Member Author

@Hixie After the roll I've checked the benchmarks and filed #20690. The suspicion so far seems this was caused by engine change flutter/engine@149ea7b (see the issue)

@github-actions github-actions bot locked as resolved and limited conversation to collaborators Aug 10, 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.

7 participants