Add build option to force inclusion of dev_dependencies#171445
Add build option to force inclusion of dev_dependencies#171445hig-dev wants to merge 2 commits intoflutter:masterfrom
Conversation
|
Thank you for your contribution. I'm really glad you're excited about Flutter! |
For future reference, excluding dev_dependencies was never fully implemented for iOS/macOS. They were still being included in the bundle, even before #171015. They just weren't being imported into the project. I added a note to our tracking issue (#163874), with what I believe the intention of this PR was. @hig-dev Feel free to comment on #163874 with your feedback |
In the latest stable release version of Flutter dev_dependencies are excluded from release builds. This leads to the issues #170119 and #167377. The PR #171015 fixed this issues only for iOS and macOS build by going a step backward. The PR changed the behavior to always include the dev_dependencies even in release build. This has the downside that the release build gets bigger again and maybe it could lead to other problems as well. So, in general it is desireable to strip out the dev_dependencies in release builds. However, there are scenarios like integration testing in which you want to have a release build including dev_dependencies.
So my PR adds the build option --force-include-dev-dependencies that ensures that the dev_dependencies are included in all build modes.
I would prefer this PR over #171015 because it does not go a step backward and solves the same issues. This PR is not a breaking change.
Example usage:
flutter build ios integration_test/foo_test.dart --release --force-include-dev-dependenciesPre-launch Checklist
///).If you need help, consider asking for advice on the #hackers-new channel on Discord.