docs icon indicating copy to clipboard operation
docs copied to clipboard

Conflicting statements regarding which MSBuild/VS versions are supported by 8.0 SDKs

Open ssrmm opened this issue 1 year ago • 4 comments

Describe the issue or suggestion

From what I could find there are at least three places that state the minimum MSBuild/VS versions supported by each feature band of the .NET SDK.

All of these places disagree on what is compatible and what isn't. Below are the compatible MSBuild/VS version according to each.

SDK Version Documentation page Download page (as of release 8.0.8) Breaking changes
8.0.1xx 17.7 17.8 17.7
8.0.2xx 17.8 N/A 17.8
8.0.3xx 17.8 17.10 17.8
8.0.4xx 17.8 17.11 17.9

After "falling in the trap" of believing the documentation page and getting a really poor error message from VS 17.8 with SDK 8.0.400, I believe that the information on the breaking changes page is the one that is correct. but I don't really know.

These version support tables should be unified to contain the correct information in all places.

ssrmm avatar Sep 24 '24 09:09 ssrmm

@marcpopMSFT Can you comment on this discrepancy?

gewarren avatar Oct 07 '24 21:10 gewarren

The download page is listing the version of VS that we ship in. It might be worth changing the wording there to make that more clear. This matches the second column in the documentation page that was linked.

The documentation page captures our intent. However, NuGet took an upgraded dependency that forced us to upgrade our minimum VS version to 17.9 in 8.0.4xx. I didn't update the documentation page but rather got NuGet to fix that upgrade and we're in the process of resolving this so that the November release will again allow for 17.8. We'll need to fix the breaking changes page at that time.

Note that even though we allow 8.0.4xx to be used with 17.8, we cannot guarantee that everything will work. The CodeStyle analyzer for example will not work in that configuration unless you set an additional property BuildWithNetFrameworkHostedCompiler set to true which is not well known. That property is on by default in net9 fwiw.

https://github.com/dotnet/sdk/pull/43995 will fix the min version

marcpopMSFT avatar Oct 08 '24 21:10 marcpopMSFT

@marcpopMSFT Regarding the download page: I was looking at these two pieces of information: Image

The Included in value seems pretty clear to me as to what it means. Are you saying that Visual Studio support is also supposed to be the version it is included in? If so, what is the reason for having what seems to be the same information twice?

ssrmm avatar Oct 10 '24 08:10 ssrmm

Ahh, the Included in section doesn't exist for the 8.0.10 release and I didn't expand the page. I think those basically mean the same thing though I'd need someone more familiar with the intent on the download page to comment. @gewarren do you know why Included in isn't there for 8.0.10 and do you know what the intent was for that versus the VS support node?

marcpopMSFT avatar Oct 10 '24 16:10 marcpopMSFT

Sorry for the delay on this. Tagging @mairaw for the downloads page question here.

Image

gewarren avatar Oct 21 '24 19:10 gewarren

Tagging @rbhanda since this data is driven from the releases.json files on dotnet/core. Rahul, did we miss any VS tagging for the latest runtime release? I'd imagine that one of those versions is included in VS.

mairaw avatar Oct 21 '24 19:10 mairaw

https://github.com/dotnet/core/pull/9558 to fix this issue

rbhanda avatar Oct 21 '24 20:10 rbhanda

I think there's still an outstanding question about the difference between Included in and Visual Studio support.

gewarren avatar Oct 21 '24 20:10 gewarren

Closing this issue as fixed. Please open a new issue if there's something else that should be changed.

gewarren avatar Jul 25 '25 23:07 gewarren