installer icon indicating copy to clipboard operation
installer copied to clipboard

Why are Microsoft.DiaSymReader.Native.* in .NET 5.0 SDK not under .NET Library License?

Open KalleOlaviNiemitalo opened this issue 5 years ago • 5 comments

License information for .NET on Windows says:

The following binaries are licensed with the Visual Studio 2019 License (not as a "trial"):

  • vcruntime140_cor3.dll (used by WPF)
  • Microsoft.DiaSymReader.Native.{x86|amd64|arm|arm64}.dll (used by .NET SDK)

However, it seems to me that Microsoft.DiaSymReader.Native.*.dll files are also being distributed in the Microsoft.DiaSymReader.Native NuGet package, licensed with the .NET Library License. So, I am wondering why the Visual Studio 2019 License was chosen for those files in the .NET 5.0 SDK. Because the full-use (not trial) terms of the Visual Studio 2019 License start with "When you acquire a valid license and either enter a product key or sign in to the software", it might be clearer to use the .NET Library License terms, which do not have "product key or sign in" requirements.

Related to https://github.com/dotnet/installer/issues/7043.

KalleOlaviNiemitalo avatar Nov 11 '20 08:11 KalleOlaviNiemitalo

Is this because Microsoft.DiaSymReader.Native/1.7.0 on NuGet supports x86, amd64, and arm, but not arm64?

KalleOlaviNiemitalo avatar Nov 11 '20 08:11 KalleOlaviNiemitalo

@jamshedd license question of VS components. Where should we route this?

marcpopMSFT avatar Nov 18 '20 22:11 marcpopMSFT

@richlander is the right person to answer why we took a dependency on the VS copy of the file rather than the NuGet one (or at least should be able to redirect)

jamshedd avatar Nov 18 '20 23:11 jamshedd

https://www.nuget.org/packages/Microsoft.DiaSymReader.Native/16.9.0-beta1.21055.5 now claims to support arm64 as well. Its source repository https://github.com/dotnet/roslyn-debug does not seem to be public yet.

KalleOlaviNiemitalo avatar Jan 09 '21 21:01 KalleOlaviNiemitalo

There was a question about (among other things) this on Hacker News: https://news.ycombinator.com/item?id=35985832

@richlander can you clarify?

reflectronic avatar May 18 '23 14:05 reflectronic