Convert boolean option to function delegate, defaulting to CliPlatform::useExe

Author: loic-peron-inetum-public

lib/src/standalone.dart

@@ -33,9 +33,10 @@ import 'utils.dart';
 final standaloneName = InternalConfigVariable.fn<String>(() => name.value);
 
 /// Whether to build native exe for the current platform instead of AOT snapshot.
+/// Defined as a function of a [CliPlatform] to allow different behavior on different platforms.
 ///
-/// This defaults to `false`.
-final useExe = InternalConfigVariable.value(false);
+/// This defaults to `CliPlatform::useExe`.
+final useExe = InternalConfigVariable.value<bool Function(CliPlatform)>((CliPlatform platform) => platform.useExe);
 
 /// For each executable entrypoint in [executables], builds a script snapshot
 /// to `build/${executable}.snapshot`.
@@ -79,7 +80,7 @@ void _compileNative() {
       existingSnapshots[path] = name;
       run('dart', arguments: [
         'compile',
-        CliPlatform.current.useExe || useExe.value ? 'exe' : 'aot-snapshot',
+        useExe.value(CliPlatform.current) ? 'exe' : 'aot-snapshot',
         path,
         for (var entry in environmentConstants.value.entries)
           '-D${entry.key}=${entry.value}',
@@ -172,15 +173,15 @@ Future<void> _buildPackage(CliPlatform platform) async {
   var archive = Archive()
     ..addFile(fileFromString("$standaloneName/src/LICENSE", await license));
 
-  if (!(platform.useExe || (useExe.value && platform.isCurrent))) {
+  if (!(platform.useNative && useExe.value(platform))) {
     archive.addFile(fileFromBytes(
         "$standaloneName/src/dart${platform.binaryExtension}",
         await _dartExecutable(platform),
         executable: true));
   }
 
   for (var name in executables.value.keys) {
-    if (platform.useExe || (useExe.value && platform.isCurrent)) {
+    if (platform.useNative && useExe.value(platform)) {
       archive.addFile(file("$standaloneName/$name${platform.binaryExtension}",
           "build/$name.native",
           executable: true));
@@ -190,7 +191,7 @@ Future<void> _buildPackage(CliPlatform platform) async {
     }
   }
 
-  if (!(platform.useExe || (useExe.value && platform.isCurrent))) {
+  if (!(platform.useNative && useExe.value(platform))) {
     // Do this separately from adding entrypoints because multiple executables
     // may have the same entrypoint.
     for (var name in executables.value.keys) {