project-system
project-system copied to clipboard
dotnet
Show MSBuild project SDKs in the Dependencies node
When referencing a custom SDK project like this:
<Project Sdk="MyCompany.MyProduct.SDK/2018.1.0-preview3">
...
it doesn't show under the Dependencies node in Visual Studio.
And also it is not seeable and updatable from NuGet Package Manager.
Also see: https://github.com/Microsoft/MSBuildSdks/issues/24
Just as a side note, that SDKs node is actually something different; "Extension SDKs" not be confused with "MSBuild SDKs". I think we should rename the former from "SDK -> Extension SDKs".
Restating some relevant info from #2248:
-
For MSBuild SDKs coming from the .NET Core SDK, they all have Version=$(NETCoreSDKVersion). There was some talk in #2248 of starting by showing $(NETCoreSDKVersion) somewhere.
-
To actually enumerate all imported MSBuild SDKs and their versions (for example, including those pulled down with NuGet MSBuild SDK resolver), I believe there will need to be a new MSBuild API added. cc @rainersigwald @livarcocc
We now have the project import tree, which may be a good place to show such a version.
For example, we could show the value of $(NETCoreSDKVersion) on top-level Sdk.props and Sdk.targets tree items.
Is the full path to the Sdk.props/Sdk.targets visible somewhere in that tree? If so, there is the version in the path. I'm not saying this is the best way to present it, but it would mean it's at least something you could find now if the path to imports can be discovered.
We now have the project import tree,
Should I expect to see this in dogfood build already? I don't.
Is the full path to the Sdk.props/Sdk.targets visible somewhere in that tree?
Not yet, but the plan is to make it available in the properties panel. Best you can do now is double-click the file then see the path in the editor tab tool tip.
Should I expect to see this in dogfood build already? I don't.
Yep, but you have to turn on 'Show all files' for the project. Feedback appreciated.
Nice!
So even if we don't do this feature (still think we should), I now have a way to find the version:
Can you see any issues with using $(NETCoreSDKVersion)? For example if using Microsoft.NET.Sdk.WindowsDesktop? I haven't played around with it yet.
Is there a general purpose version that comes with the MSBuild SDK itself (think like when the SDK comes from a package) that we can pull so we don't have to specifically handle this variable that is SDK-specific?
SDK resolvers return a version to MSBuild so I think the right, general way would be to ask msbuild for a list of resolved sdks and their names and versions. (# 2 in https://github.com/dotnet/project-system/issues/3463#issuecomment-513947382)
I would imagine it isn’t a huge change for msbuild to expose resolved sdks, but I could be very wrong. I think we should investigate what that would take.
The source of truth is what the sdk resolver returns to msbuild. The package resolver returns a package version. The .NET core sdk resolver returns the .NET Core SDK version for all sdks it bundles.