-
Notifications
You must be signed in to change notification settings - Fork 50
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
[native_assets_cli] Add config.code.flutter.cApi
#1836
Comments
I don't think we'd want anything in dart-lang/native for that, as that is flutter specific. So it would be a flutter-specific extension to the protocol. Because of that it would probably live in flutter/flutter. In fact it would probably be a package that flutter ships as part of Flutter SDK (i.e. a pined Regarding stability of
The => I'd tend to say flutter specifics can be in |
Agreed. (Seems like I violated my own rule and filed the issue in the wrong repo!)
So you're kind of saying that extensibility should only be at the root of the config.json, not in nested maps? (I just filed #1837.) |
Is the Originally posted by @mkustermann in #1837 (comment) @stuartmorgan Could you elaborate on the need for having |
Developers might want to build native code against
flutter.h
.To prevent version skew between the package and the Flutter SDK, from-source build should use
flutter.h
from Flutter itself.We should provide a path to it in the build config. Something along the lines of
(Should flutter-specific code-specific config live under
config.flutter.code
orconfig.code.flutter
?)This is analogous to:
To support prebuilt binaries, the
version
should be taken into account.Note that this doesn't solve version skew between generated
dart:ffi
bindings with FFIgen on aflutter.h
and thenflutter.h
getting a breaking change. This could theoretically be solved by providing access toversion
in Dart (for example via passing inversion
as a define in a C lib and compiling it in and accessing it via an FFI call, or by writing the version to a data asset and accessing it at runtime), and then branching between two sets of generated bindings. (This is a similar issue to #1589.)The text was updated successfully, but these errors were encountered: