Don't default to standard output paths based on target framework

Author: dsplaisted

src/Tasks/Microsoft.NET.Build.Tasks/targets/Microsoft.NET.DefaultOutputPaths.targets

@@ -25,13 +25,17 @@ Copyright (c) .NET Foundation. All rights reserved.
     <AppendTargetFrameworkToOutputPath Condition="'$(AppendTargetFrameworkToOutputPath)' == ''">true</AppendTargetFrameworkToOutputPath>
   </PropertyGroup>
 
-  <!-- Set UseStandardOutputPaths if not set and TargetFramework >= .NET 8 -->
+  <!-- NOTE: If we want to default UseStandardOutputPaths to true when targeting a given version of .NET or higher, this is where we would do it.
+
+    It would look something like this:
+
   <PropertyGroup Condition="'$(UseStandardOutputPaths)' == '' and
                             '$(TargetFrameworks)' == '' and
                             '$(TargetFrameworkIdentifier)' == '.NETCoreApp' and
                             $([MSBuild]::VersionGreaterThanOrEquals($(TargetFrameworkVersion), 8.0))">
     <UseStandardOutputPaths>true</UseStandardOutputPaths>
   </PropertyGroup>
+  -->
 
   <!-- Handle RootOutputPath and if it wasn't set in Sdk.props -->
   <PropertyGroup Condition="'$(RootOutputPath)' != ''">

src/Tests/Microsoft.NET.TestFramework/OutputPathCalculator.cs

@@ -118,36 +118,41 @@ private bool UsesStandardOutputPaths()
                 return UseStandardOutputPaths.Value;
             }
 
-            if (!IsSdkProject)
-            {
-                return false;
-            }
-
-            string targetFramework = TryGetTargetFramework();
-
-            if (targetFramework == null)
-            {
-                return false;
-            }
-
-            if (IsMultiTargeted())
-            {
-                return false;
-            }
+            
+
+            //  If we end up enabling standard output paths when targeting a certain version of .NET or higher, the logic for the rest of this method would look something like this:s
+            //if (!IsSdkProject)
+            //{
+            //    return false;
+            //}
+
+            //string targetFramework = TryGetTargetFramework();
+
+            //if (targetFramework == null)
+            //{
+            //    return false;
+            //}
+
+            //if (IsMultiTargeted())
+            //{
+            //    return false;
+            //}
+
+            //var framework = NuGetFramework.Parse(targetFramework);
+            //if (framework.Framework != ".NETCoreApp")
+            //{
+            //    return false;
+            //}
+            //if (framework.Version.Major >= 8)
+            //{
+            //    return true;
+            //}
+            //else
+            //{
+            //    return false;
+            //}
 
-            var framework = NuGetFramework.Parse(targetFramework);
-            if (framework.Framework != ".NETCoreApp")
-            {
-                return false;
-            }
-            if (framework.Version.Major >= 8)
-            {
-                return true;
-            }
-            else
-            {
-                return false;
-            }
+            return false;
         }
 
         public string GetOutputDirectory(string targetFramework = null, string configuration = "Debug", string runtimeIdentifier = "")