Skip MSBuild and call only CSC for simple file-based apps

Directory.Packages.props

@@ -18,6 +18,7 @@
     <PackageVersion Include="Microsoft.Build.NuGetSdkResolver" Version="$(MicrosoftBuildNuGetSdkResolverPackageVersion)" />
     <PackageVersion Include="Microsoft.CodeAnalysis" Version="$(MicrosoftCodeAnalysisPackageVersion)" />
     <PackageVersion Include="Microsoft.CodeAnalysis.Analyzer.Testing" Version="$(MicrosoftCodeAnalysisAnalyzerTestingVersion)" />
+    <PackageVersion Include="Microsoft.CodeAnalysis.BuildClient" Version="5.0.0-1.25317.1" />
     <PackageVersion Include="Microsoft.CodeAnalysis.CSharp" Version="$(MicrosoftCodeAnalysisCSharpPackageVersion)" />
     <PackageVersion Include="Microsoft.CodeAnalysis.CSharp.CodeStyle" Version="$(MicrosoftCodeAnalysisPackageVersion)" />
     <PackageVersion Include="Microsoft.CodeAnalysis.CSharp.Features" Version="$(MicrosoftCodeAnalysisCSharpPackageVersion)" />

NuGet.config

@@ -32,6 +32,7 @@
     <add key="richnav" value="https://pkgs.dev.azure.com/azure-public/vside/_packaging/vs-buildservices/nuget/v3/index.json" />
     <!-- mstest dependencies -->
     <add key="test-tools" value="https://pkgs.dev.azure.com/dnceng/public/_packaging/test-tools/nuget/v3/index.json" />
+    <add key="temporary" value="https://pkgs.dev.azure.com/dnceng/public/_packaging/general-testing/nuget/v3/index.json" />
   </packageSources>
   <disabledPackageSources>
     <clear />

src/Cli/Microsoft.DotNet.Cli.Utils/ArgumentEscaper.cs

@@ -86,12 +86,12 @@ private static IEnumerable<string> EscapeArgArrayForCmd(IEnumerable<string> argu
         return escapedArgs;
     }
 
-    public static string EscapeSingleArg(string arg)
+    public static string EscapeSingleArg(string arg, Func<string, bool>? additionalShouldSurroundWithQuotes = null)
     {
         var sb = new StringBuilder();
 
         var length = arg.Length;
-        var needsQuotes = length == 0 || ShouldSurroundWithQuotes(arg);
+        var needsQuotes = length == 0 || ShouldSurroundWithQuotes(arg) || additionalShouldSurroundWithQuotes?.Invoke(arg) == true;
         var isQuoted = needsQuotes || IsSurroundedWithQuotes(arg);
 
         if (needsQuotes) sb.Append("\"");

src/Cli/Microsoft.DotNet.Cli.Utils/Product.cs

@@ -5,18 +5,32 @@
 
 namespace Microsoft.DotNet.Cli.Utils;
 
-public class Product
+public static class Product
 {
     public static string LongName => LocalizableStrings.DotNetSdkInfo;
-    public static readonly string Version = GetProductVersion();
+    public static readonly string Version;
+    public static readonly string TargetFrameworkVersion;
 
-    private static string GetProductVersion()
+    static Product()
     {
         DotnetVersionFile versionFile = DotnetFiles.VersionFileObject;
-        return versionFile.BuildNumber ??
+        Version = versionFile.BuildNumber ??
                 System.Diagnostics.FileVersionInfo.GetVersionInfo(
                         typeof(Product).GetTypeInfo().Assembly.Location)
                     .ProductVersion ??
                 string.Empty;
+
+        int firstDotIndex = Version.IndexOf('.');
+        if (firstDotIndex >= 0)
+        {
+            int secondDotIndex = Version.IndexOf('.', firstDotIndex + 1);
+            TargetFrameworkVersion = secondDotIndex >= 0
+                ? Version.Substring(0, secondDotIndex)
+                : Version;
+        }
+        else
+        {
+            TargetFrameworkVersion = string.Empty;
+        }
     }
 }

src/Cli/dotnet/Commands/CliCommandStrings.resx

@@ -1539,9 +1539,13 @@ Tool '{1}' (version '{2}') was successfully installed. Entry is added to the man
     <comment>{0} is an option name like '--no-build'.</comment>
   </data>
   <data name="NoBinaryLogBecauseUpToDate" xml:space="preserve">
-    <value>Warning: Binary log option was specified but build will be skipped because output is up to date, specify '--no-cache' to force build.</value>
+    <value>Warning: Binary log option was specified but build will be skipped because output is up to date. Specify '--no-cache' to force build.</value>
     <comment>{Locked="--no-cache"}</comment>
   </data>
+  <data name="NoBinaryLogBecauseRunningJustCsc" xml:space="preserve">
+    <value>Warning: Binary log option was specified but MSBuild will be skipped because running just csc is enough. Specify '--no-cache' to force full build.</value>
+    <comment>{Locked="--no-cache"}{Locked="MSBuild"}{Locked="csc"}</comment>
+  </data>
   <data name="PublishAppDescription" xml:space="preserve">
     <value>Publisher for the .NET Platform</value>
   </data>