Remove harvesting of M.E.DependencyModel

[ViktorHofer]

The netstandard1.6 configuration of Microsoft.Extensions.DependencyModel
isn't built anymore. Instead the already built matching binary from the
latest available package version is redistributed when packaging the
DependencyModel library.

Also dropping the netstandard1.3 asset and the net451 one as the
minimum supported set of platforms are ones that support netstandard2.0.

In addition to the harvesting removal, cleaning up the src project which
had an unnecessary condition and property set.

Contributes to #47530

cc @terrajobst @ericstj

[ghost]

Tagging subscribers to this area: @eerhardt
See info in area-owners.md if you want to be subscribed.

Issue Details

---

The netstandard1.6 configuration of Microsoft.Extensions.DependencyModel
isn't built anymore. Instead the already built matching binary from the
latest available package version is redistributed when packaging the
DependencyModel library.

Also dropping the netstandard1.3 asset and the net451 one as the
minimum supported set of platforms are ones that support netstandard2.0.

In addition to the harvesting removal, cleaning up the src project which
had an unnecessary condition and property set.

Contributes to #47530

cc @terrajobst @ericstj

Author:	ViktorHofer
Assignees:	-
Labels:	area-DependencyModel
Milestone:	-

[eerhardt]

Thanks @ViktorHofer !

[ViktorHofer]

.packages\microsoft.dotnet.build.tasks.packaging\6.0.0-beta.21216.2\build\Packaging.targets(1119,5): error : (NETCORE_ENGINEERING_TELEMETRY=Build) Microsoft.Extensions.DependencyModel should be supported on .NETCoreApp,Version=v1.0/win7-x86 but has no compile assets.

@ericstj what tells the packaging system that the lib should be supported on netcoreapp1.0? I'm missing some context here.

[ericstj]

IIRC it's looking at the previous package and determining what it supported when deciding what the new package should support. You should be able to repro relatively cheaply locally by building the pkgproj and examining the binlog.

cleaning up the src project which had an unnecessary condition and property set.

Might be other places to clean that up: #39099 (comment)

[ericstj]

Looks like this now has a merge conflict.

[ViktorHofer]

The PR is ready. I'm now using the new packaging hook to exclude frameworks from validation which aren't supported anymore.

[ghost]

Hello @ViktorHofer!

Because this pull request has the auto-merge label, I will be glad to assist with helping to merge this pull request once all check-in policies pass.

p.s. you can customize the way I help with merging this pull request, such as holding this pull request until a specific person approves. Simply @mention me (@msftbot) and give me an instruction to get started! Learn more here.

[ViktorHofer]

Network flakiness:

  Retrying (Retry(total=0, connect=None, read=None, redirect=None, status=None)) after connection broken by 'NewConnectionError('<pip._vendor.urllib3.connection.VerifiedHTTPSConnection object at 0x7f8f81b940>: Failed to establish a new connection: [Errno -3] Temporary failure in name resolution',)': /simple/certifi/
  Could not find a version that satisfies the requirement certifi==2020.6.20 (from -r /root/helix/scripts/runtime_python_requirements.txt (line 1)) (from versions: )
No matching distribution found for certifi==2020.6.20 (from -r /root/helix/scripts/runtime_python_requirements.txt (line 1))