Skip to content

Enable the simultaneous use of Dart plugins and custom Dart entrypoints #228

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

Merged
merged 1 commit into from
Sep 7, 2021

Conversation

swift-kim
Copy link
Member

@swift-kim swift-kim commented Sep 3, 2021

Find entry points functions from the main dart file and inject to the generated main file.

Resolves the limitation mentioned in #171 ("The use of custom entrypoints may break ...").

Note that this change doesn't take account of all possible uses of the vm:entry-point pragma described in https://github.com/dart-lang/sdk/blob/master/runtime/docs/compiler/aot/entry_point_pragma.md#syntax. Only functions are allowed and the second parameter of @pragma is not honored.

@jkpu @pkosko

See also: The AST analyzer (https://github.com/dart-lang/sdk/blob/master/pkg/analyzer/doc/tutorial/ast.md)

@swift-kim swift-kim force-pushed the fix-service-entrypoints branch from d508f1c to 35414f0 Compare September 3, 2021 12:02
@swift-kim swift-kim force-pushed the fix-service-entrypoints branch from 35414f0 to 101d472 Compare September 3, 2021 12:07
@swift-kim swift-kim requested a review from a team September 6, 2021 01:33
Copy link
Contributor

@bbrto21 bbrto21 left a comment

Choose a reason for hiding this comment

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

I tested using Dart plugin, it works fine.
LGTM

@swift-kim swift-kim merged commit 62dbcf3 into flutter-tizen:master Sep 7, 2021
swift-kim added a commit to swift-kim/flutter-tizen that referenced this pull request Oct 20, 2021
swift-kim added a commit that referenced this pull request Oct 21, 2021
* Update Dart plugin registrant

* Clean up

* Remove TizenAnalyzeCommand

* Clean up tizen_plugins.dart and expose renderTemplateToFile

* Update test.dart

* Additional clean-up

* Revert "Enable the simultaneous use of Dart plugins and custom Dart entrypoints (#228)"

This reverts commit 62dbcf3.

* Reimplement multiple Dart entrypoints support

* Remove analyzer from dependencies
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants