You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
@dcharkes is pana going to run the native-assets build script to get this information, or is it something that is hardcoded in pubspec.yaml under platforms.android?
I think going with the hook is ok. I don't see the need for it in pubspec.yaml, and as you say, it can get out of sync with the actual platform support.
Pana is sandboxed, and should(??) be ok running user code. But that would be something new for us to do.
Activity
sigurdm commentedon Sep 3, 2024
Is this really an issue for the site? (dart-lang/pub-dev)?
dcharkes commentedon Sep 3, 2024
The reporting is? (I'll file another issue on the SDK for reasoning about SDK native version constraints in general.)
jonasfj commentedon Sep 3, 2024
@dcharkes is
panagoing to run the native-assets build script to get this information, or is it something that is hardcoded inpubspec.yamlunderplatforms.android?dcharkes commentedon Sep 3, 2024
Those are two options.
If we want the solution to work for Flutter plugins, and Dart packages only using
dart:ffi, thepubspec.yaml.If we don't care about forcing those users to use a build hook, we could also make the hook report it. See the linked issues.
sigurdm commentedon Sep 5, 2024
I think going with the hook is ok. I don't see the need for it in pubspec.yaml, and as you say, it can get out of sync with the actual platform support.
Pana is sandboxed, and should(??) be ok running user code. But that would be something new for us to do.