[native_assets] Support `dart pub global run`

[lindeer]

now my package contains a mylib-exe.dart in bin directory and in my pubspec.yaml:

environment:
  sdk: ^3.3.0

executables:
  mylib-exe:

dependencies:

but after I install dart pub global activate mylib both --source git or --source path, and run mylib-exe in cmd, report error:

Invalid argument(s): Couldn't resolve native function 'mylib_open' in 'package:mylib/src/lib_mylib.dart' : No asset with id 'package:mylib/src/lib_mylib.dart' found. No available native assets. Attempted to fallback to process lookup. /opt/programs/flutter/bin/cache/dart-sdk/bin/dart: undefined symbol: mylib_open.

seems build script not triggered and .so file not generated, I edit mylib-exe in ~/.pub-cache/bin/mylib-exe:

--- /home/wesley/.pub-cache/bin/mylib	2024-06-19 16:19:36.250444945 +0800
+++ mylib	2024-06-19 16:19:25.571916616 +0800
@@ -12,7 +12,7 @@
   if [ $exit_code != 253 ]; then
     exit $exit_code
   fi
-  dart pub global run mylib:mylib "$@"
+  dart pub global run --enable-experiment=native-assets mylib:mylib "$@"
 else
-  dart pub global run mylib:mylib "$@"
+  dart pub global run --enable-experiment=native-assets mylib:mylib "$@"
 fi

still not work. How should I fix it, thanks~

[dcharkes]

Thanks for the report @lindeer!

We don't have support yet for pub global run.

I would have expected the error message to complain that the experiment is required instead of run as if there are no native assets.

Just to double check, everything works fine when doing dart run --enable-experiment=native-assets mylib:mylib?

(Also, transferring this to the Dart SDK repo, as it should be implemented there.)

[dcharkes]

FYI @jonasfj @sigurdm

[dcharkes]

@jonasfj @sigurdm What would be a good way to save the binaries for pub global activated packages produced by the build hook.

And I'm thinking, it's probably cleanest to already invoke the build hook to produce the native assets during dart pub global activate and not during dart pub global run. (But we don't precompile the Dart code on activate judging from shell script in bin/.)

[lindeer]

Just to double check, everything works fine when doing dart run --enable-experiment=native-assets mylib:mylib?

yeah, everything is OK when I run dart run.

[sigurdm]

And I'm thinking, it's probably cleanest to already invoke the build hook to produce the native assets during dart pub global activate and not during dart pub global run. (But we don't precompile the Dart code on activate judging from shell script in bin/.)

We do precompile - except for when the package is global activated from path.

We also support recompilation if the snapshot was compiled with a different VM.

[dcharkes]

After some offline discussion with @jonasfj and @sigurdm, this should probably be addressed by dart pub (package:pub) no longer manually compiling Dart to jit snapshots, but by dart pub invoking dart build exe (package:dartdev) so that dartdev is responsible for building dart code and bundling native assets. #60889

(Instead of trying to patch what pub is currently doing.)