Fix handle spaces in the path to the cl.exe compiler

When the Visual Studio C++ compiler (cl.exe) is installed in a path
containing spaces (e.g., the default "C:\Program Files\Microsoft Visual
Studio\2022\..."), the build system failed to execute it correctly.

The command was invoked without proper quoting, leading to an error:
  'C:\Program' is not recognized as an internal or external command

Signed-off-by: Dee HY <[email protected]>

Author: dongfengweixiao

pkgs/native_toolchain_c/lib/src/cbuilder/cbuilder.dart

@@ -79,6 +79,7 @@ class CBuilder extends CTool implements Builder {
     super.linkModePreference,
     super.optimizationLevel = OptimizationLevel.o3,
     this.buildMode = BuildMode.release,
+    super.runInShell,
   }) : super(type: OutputType.library);
 
   CBuilder.executable({
@@ -105,6 +106,7 @@ class CBuilder extends CTool implements Builder {
     super.cppLinkStdLib,
     super.optimizationLevel = OptimizationLevel.o3,
     this.buildMode = BuildMode.release,
+    super.runInShell,
   }) : super(
          type: OutputType.executable,
          assetName: null,
@@ -203,6 +205,7 @@ class CBuilder extends CTool implements Builder {
       language: language,
       cppLinkStdLib: cppLinkStdLib,
       optimizationLevel: optimizationLevel,
+      runInShell: runInShell,
     );
     await task.run();
 

pkgs/native_toolchain_c/lib/src/cbuilder/clinker.dart

@@ -41,6 +41,7 @@ class CLinker extends CTool implements Linker {
     super.cppLinkStdLib,
     super.linkModePreference,
     super.optimizationLevel = OptimizationLevel.o3,
+    super.runInShell,
   }) : super(type: OutputType.library);
 
   /// Runs the C Linker with on this C build spec.
@@ -94,6 +95,7 @@ class CLinker extends CTool implements Linker {
       language: language,
       cppLinkStdLib: cppLinkStdLib,
       optimizationLevel: optimizationLevel,
+      runInShell: runInShell,
     );
     await task.run();
 

pkgs/native_toolchain_c/lib/src/cbuilder/ctool.dart

@@ -166,6 +166,12 @@ abstract class CTool {
   /// What optimization level should be used for compiling.
   final OptimizationLevel optimizationLevel;
 
+  /// Whether to run the process in a shell.
+  ///
+  /// If the path to cl.exe contains spaces, this option needs to be set to
+  /// false.
+  final bool? runInShell;
+
   CTool({
     required this.name,
     required this.packageName,
@@ -186,5 +192,6 @@ abstract class CTool {
     required this.linkModePreference,
     required this.type,
     required this.optimizationLevel,
+    this.runInShell,
   });
 }

pkgs/native_toolchain_c/lib/src/cbuilder/run_cbuilder.dart

@@ -50,6 +50,7 @@ class RunCBuilder {
   final Language language;
   final String? cppLinkStdLib;
   final OptimizationLevel optimizationLevel;
+  final bool? runInShell;
 
   RunCBuilder({
     required this.input,
@@ -73,6 +74,7 @@ class RunCBuilder {
     this.language = Language.c,
     this.cppLinkStdLib,
     required this.optimizationLevel,
+    this.runInShell,
   }) : outDir = input.outputDirectory,
        assert(
          [executable, dynamicLibrary, staticLibrary].whereType<Uri>().length ==
@@ -196,6 +198,7 @@ class RunCBuilder {
         captureOutput: false,
         throwOnUnexpectedExitCode: true,
         environment: environment,
+        runInShell: runInShell,
       );
     } else {
       await _compile(
@@ -343,6 +346,7 @@ class RunCBuilder {
       logger: logger,
       captureOutput: false,
       throwOnUnexpectedExitCode: true,
+      runInShell: runInShell,
     );
   }
 
@@ -401,6 +405,7 @@ class RunCBuilder {
       captureOutput: false,
       stdoutLogLevel: Level.INFO,
       throwOnUnexpectedExitCode: true,
+      runInShell: runInShell,
     );
 
     if (staticLibrary != null) {

pkgs/native_toolchain_c/lib/src/utils/run_process.dart

@@ -22,6 +22,7 @@ Future<RunProcessResult> runProcess({
   Level stdoutLogLevel = Level.FINE,
   int expectedExitCode = 0,
   bool throwOnUnexpectedExitCode = false,
+  bool? runInShell,
 }) async {
   final printWorkingDir =
       workingDirectory != null && workingDirectory != Directory.current.uri;
@@ -41,7 +42,7 @@ Future<RunProcessResult> runProcess({
     arguments,
     workingDirectory: workingDirectory?.toFilePath(),
     environment: environment,
-    runInShell: Platform.isWindows && workingDirectory != null,
+    runInShell: runInShell ?? Platform.isWindows && workingDirectory != null,
   );
 
   final stdoutSub = process.stdout.listen((List<int> data) {