Nullref in .targets file

Description

paket update with Suave’s faulty spec https://github.com/SuaveIO/suave/issues/638 causes paket to assume .net core, despite framework: net461 being a given constraint in paket.dependencies. This in turn causes, “Object reference not set to an instance of an object” to be thrown and the build to fail when running xbuild Project.sln.

Expected behavior

Not to throw null ref exceptions. Not to add .Net core dependencies when the framework target is constrained.

Actual behavior

Throwing null ref.

Known workarounds

Manually remove the paket.targets import that was added.

[haf]

If this https://github.com/haf/expecto/pull/175 merges, I’ll be moving all my projects to FAKE.

[matthid]

That said, netstandard should be the default, pcl are deprecated.

Yes exactly

For nuget client is easier to maintain compatibility, old version doesnt auto-update, so doesnt consider the netstandard.

You can use strategy: min

The root issue is NETStandard.Library world vs old world with pcl only. Is a choice an user can do. @matthid maybe adding a strategy modifier, can help to disambiguate

I mean yes we could prefer pcl in that case (and add some option around it) but I don’t think that is a good idea going forward. I still don’t get why netstandard dlls wouldn’t work at runtime. There might be a different issue here than paket adding some hacks to prevent netstandard dlls.