Fixes #3012

[msatina]

fix #3012

Get TargetFramework(s) from runtime as a fallback in case they are not defined in csproj file.

[enricosada]

seems ok (i need to test it locally later)

@msatina awesome work, can you add an integration test?

like

Paket/integrationtests/Paket.IntegrationTests/RestoreSpecs.fs

Lines 87 to 99 in 680bc18

	[<Test>]
	let ``#3000-a dotnet restore``() =
	let scenario = "i003000-netcoreapp2"
	let projectName = "c1"
	let packageName = "AutoMapper"
	let workingDir = scenarioTempPath scenario
	let projectDir = workingDir @@ projectName
	[ packageName; (packageName.ToLower()) ] |> Seq.iter clearPackage
	prepareSdk scenario
	directDotnet false "restore" projectDir |> ignore
	directDotnet false "build --no-restore" projectDir |> ignore

for no TargetFramework in csproj, but in Directory.Build.props

[enricosada]

as a note, VS need to fix this scenario probably #2863 (comment)

[matthid]

Note that calling paket restore is by design here. We tweaked paket restore for maximum performance (including skipped command line parsing). With this change restore will most likely be a lot slower if stuff was already restored.

[matthid]

What I'm trying to say is that we either need a different solution / carefully tweak performance for this case or some other trick

[forki]

ping

[forki]

@matthid @enricosada better?

[forki]

thanks!