update F# templates to no longer unconditionally override FSharpTargetsPath. by 0x53A · Pull Request #2575 · dotnet/fsharp

@enricosada did it this way to get all the right SDK bits and pieces - targets etc. - into place, I'm not sure

as a note, for new sdk, i dont use the target file from vf# repo. i use FSharp.NET.Core.Sdk.targets, that's where CoreCompile is defined

Is there any reason we can't move both the spec of the FSharp.Compiler.Tools package

for FSharp.Compiler.Tools i dont mind. it's pratically Microsoft.FSharp.Compiler.netcore with some props file to locate fsc.
As a note, i tried initially to do exactly that with my old PR #2250 .
Is the same as having FSharp.Core package here ihmo, why not? i am just merging Microsoft.FSharp.Core.netcore too.
Also for FSharp.Build.dll having that signed from MS is better, because of internal visible stuff.

and the whole of netcorecli-fsc into this repo

Nothing techically, is all about project development.
Atm i can:

long term is a good idea to have it near FSharp.Core and FSharp.Compiler.Tools (so maybe in fsharp/fsharp too, just compiler+fsharcore+fcs? 😄 ).
But atm is a bit complicated to do so.