-
Notifications
You must be signed in to change notification settings - Fork 29.7k
Description
Right now the engine prefers some settings to the value in Info.plist even if the command line specifies a different value. This is almost certainly unexpected for developers manually specifying command line arguments.
We should change the code here https://github.com/flutter/engine/blob/95a962fea38bb27cba07edadb525857d1f712ae9/shell/platform/darwin/ios/framework/Source/FlutterDartProject.mm#L99 so that the settings are read from the command line after the plist is parsed. Unfortunately, the settings for Android are handled in a few different spots and will have to be audited differently. We also have to audit whether the tool is ever specifying a setting explicitly with the default value or not.
For now, I am changing the behavior of this for enable-impeller to try to get it in for the next stable without too much disruption.