-
Notifications
You must be signed in to change notification settings - Fork 1.7k
dart build exe
promised layout
#60730
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Comments
This PR enables native assets on the main and dev channel by default, and make native assets available on the beta channel. This PR removes the flag from invocations. The helper packages (`package:hooks` and `package:code_assets`) will stay 0.x for now, until the SDK constraint can be bumped to a beta release and we're happy with the Dart API. `dart build` is also made available as preview (without a flag on the main, dev, and beta channels). We're still finalizing the spec for this command. (#60730) `dart test` will need dart-lang/test#2501. This means users will need to update their `package:test` dependency. This PR refactors the way that invalid `package_config.json`s are handled: they are now loaded in the dartdev commands and handled there. Bug: #50565 Project: https://github.com/orgs/dart-lang/projects/99/ Change-Id: I7db9ff6d7196750cab9379a4605c6bbf89a974d7 Cq-Include-Trybots: luci.dart.try:pkg-linux-debug-try,pkg-linux-release-arm64-try,pkg-linux-release-try,pkg-mac-release-arm64-try,pkg-win-release-arm64-try,pkg-mac-release-try,pkg-win-release-try Reviewed-on: https://dart-review.googlesource.com/c/sdk/+/429920 Commit-Queue: Daco Harkes <dacoharkes@google.com> Reviewed-by: Martin Kustermann <kustermann@google.com>
I've been giving this some thought, and I think we should align Moreover, I believe we have (at least) two very different Dart apps:
Going at OS conventions, CLI apps and UI apps are treated quite differently:
It's conceivable that we might have more formats later:
So, I believe we should structure
Now, it's rather common that Having OS-compliant Having subcommands of We'll deprecate We'll deprecate I think we should hold off for building Any feedback welcome! Anything we forgot to consider? Any terminology to improve? @goderbauer @mkustermann @bkonyi @mit-mit @mosuem (I believe @stuartmorgan-g and @jmagman might have made some of these design decisions for Flutter. Feel free to cc other people who were part of designing |
My gut feeling is that the standalone Dart SDK is probably not the way to build UI apps. If someone makes a new UI framework, mots likely they want to customize the way
So I think a popular UI framework will make a custom SDK for their developers. We see that already with Also putting any UI specifics into core Dart SDK would mean the Dart team has to maintain those pieces, I don't think we want this. Very intentionally we've make the new dart hooks support a layered architecture where it's rather easy for new SDKs to build upon the basic building blocks and extend where needed. It's indeed an interesting question whether a |
Yes, I'm not convinced either we should have support for UI apps in
It's going to massively inflate the total binary size if we don't support multiple entry points but the user wants multiple entry points. Though maybe the tree-shaking currently doesn't support reasoning about multiple entry points. It's probably fine to start with only supporting a single entry-point, but the API and output format should be designed so we can have multiple entry points. |
For custom SDKs we need the SDK authors to invoke (It would be somewhat tempting to use The place where cc @bkonyi @jakemac53 @srawlins Any thoughts? (Dart CLI sync) Concrete list of changes:
Possible follow up changes:
|
just a wild idea, maybe it makes sense to describe build layouts in a yaml file and have some predefined |
Uh oh!
There was an error while loading. Please reload this page.
Current proposal: #60730 (comment)
What do we promise as layout for
dart build exe
?If we change the directory layout later, we would break users that rely on the results. So we need to specify what users can expect as an output.
cc @mkustermann @bkonyi
Prerequisite for:
dart pub global activate
to usedart build exe
pub#4313The text was updated successfully, but these errors were encountered: