Skip to content

Commit f0b5108

Browse files
authored
Make desktop the default again (#12170)
* Make desktop the default again * Fix typo
1 parent b0a5470 commit f0b5108

File tree

2 files changed

+9
-3
lines changed

2 files changed

+9
-3
lines changed

src/fsharp/FSharp.Build/Microsoft.FSharp.Targets

Lines changed: 6 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -61,6 +61,12 @@ this file.
6161
6262
If Shim helpers are present then the values are valid and set by the Microsoft.FSharp.Helpers.props
6363
-->
64+
65+
<!-- Here we set the default value to true when nothing is set, it will be removed in a future release when we default to the net sdk compiler -->
66+
<PropertyGroup Condition="'$(FSharp_Shim_Present)' == 'true'">
67+
<FSharpPreferNetFrameworkTools Condition="'$(FSharpPreferNetFrameworkTools)' == ''">true</FSharpPreferNetFrameworkTools>
68+
</PropertyGroup>
69+
6470
<PropertyGroup Condition="'$(FSharp_Shim_Present)' == 'true' and '$(FSharpPreferNetFrameworkTools)' == 'true'">
6571
<FscToolPath>$(Fsc_NetFramework_ToolPath)</FscToolPath>
6672
<FscToolExe Condition="'$(FSharpPrefer64BitTools)' != 'false'">$(Fsc_NetFramework_AnyCpu_ToolExe)</FscToolExe>

tests/fsharp/SDKTests/tests/WhichFSharpCompiler - Default Settings.proj

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -9,9 +9,9 @@
99
<PropertyGroup>
1010
<ExpectedFSharpShimPresent>true</ExpectedFSharpShimPresent>
1111
<ExpectedFSharpCompilerPath>/Common7/IDE/CommonExtensions/Microsoft/FSharp/Tools/</ExpectedFSharpCompilerPath>
12-
<ExpectedFscToolExe>dotnet.exe</ExpectedFscToolExe>
13-
<ExpectedFscToolPath>/_NetCoreRoot_/</ExpectedFscToolPath>
14-
<ExpectedDotnetFscCompilerPath>/FSharp/fsc.dll</ExpectedDotnetFscCompilerPath>
12+
<ExpectedFscToolExe>fscAnyCpu.exe</ExpectedFscToolExe>
13+
<ExpectedFscToolPath>_VsInstallRoot_/Common7/IDE/CommonExtensions/Microsoft/FSharp/Tools/</ExpectedFscToolPath>
14+
<ExpectedDotnetFscCompilerPath></ExpectedDotnetFscCompilerPath>
1515
</PropertyGroup>
1616

1717
<Import Project="ToolsTest.props" />

0 commit comments

Comments
 (0)