dart-lang / pub-dev

The pub.dev website
https://pub.dev
BSD 3-Clause "New" or "Revised" License
790 stars 146 forks source link

Surface supported architectures per OS #8010

Open dcharkes opened 2 months ago

dcharkes commented 2 months ago

Tracking issue for the surfacing on pub.dev part of:

We should consider surfacing the supported hardware architectures on pub per OS.

And there are more things we could report, such as the minimum supported SDK version for Android, iOS, and MacOS.

sigurdm commented 2 months ago

Is this really an issue for the site? (dart-lang/pub-dev)?

dcharkes commented 2 months ago

The reporting is? (I'll file another issue on the SDK for reasoning about SDK native version constraints in general.)

jonasfj commented 2 months ago

@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?

dcharkes commented 2 months ago

Those are two options.

If we want the solution to work for Flutter plugins, and Dart packages only using dart:ffi, the pubspec.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 commented 2 months ago

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.