Adding default ProduceReferenceAssembly value for F# (#13085)

* Adding default ProduceReferenceAssembly value for f# * Disabling f# test on full framework
dotnet · Aug 26, 2020 · 26855b5 · 26855b5
1 parent 041605d
commit 26855b5
Show file tree

Hide file tree

Showing 2 changed files with 20 additions and 0 deletions.
diff --git a/src/Tasks/Microsoft.NET.Build.Tasks/targets/Microsoft.NET.Sdk.FSharp.props b/src/Tasks/Microsoft.NET.Build.Tasks/targets/Microsoft.NET.Sdk.FSharp.props
@@ -28,4 +28,8 @@ Copyright (c) .NET Foundation. All rights reserved.
   </PropertyGroup>
   <Import Condition=" '$(UseBundledFSharpTargets)' == 'true' and Exists('$(FSharpPropsShim)') " Project="$(FSharpPropsShim)" />
 
+  <PropertyGroup>
+    <ProduceReferenceAssembly>false</ProduceReferenceAssembly>
+  </PropertyGroup>
+
 </Project>
diff --git a/src/Tests/Microsoft.NET.Build.Tests/GivenThatWeWantToBuildADesktopExeWithFSharp.cs b/src/Tests/Microsoft.NET.Build.Tests/GivenThatWeWantToBuildADesktopExeWithFSharp.cs
@@ -57,5 +57,21 @@ public void It_builds_a_simple_desktop_app()
                 "System.ValueTuple.dll",
             });
         }
+
+        // https://github.com/dotnet/sdk/issues/12560
+        [CoreMSBuildAndWindowsOnlyFact]
+        public void It_builds_a_simple_net50_app()
+        {
+            var testAsset = _testAssetsManager
+                .CopyTestAsset("HelloWorldFS")
+                .WithSource()
+                .WithTargetFramework("net5.0");
+
+            var buildCommand = new BuildCommand(testAsset);
+            buildCommand
+                .Execute()
+                .Should()
+                .Pass();
+        }
     }
 }