Conversation
We already require a VS 17.8 installation so the version bump shouldn't cause any min version requirement issues. This should unblock dotnet/runtime#97087
|
I'm not sure this is going to work. Since .NET 9 is being bootstrapped on itself, N-1 is really just a previous build of .NET 9. In that case, it would need to be a reference to 4.10.0. |
|
There aren't any SBRP for Microsoft.CodeAnalysis* with a 4.8.0 version which made me assume that because of that, the correct N-1 artifacts will be chosen. Let's try it out... The version upgrade makes sense anyway. |
|
Oh, that's true. It may cause a prebuilt. We can live with that to get Preview 1 out the door. |
Updating to 4.6.0 caused prebuilts that were fixed in dotnet/source-build-reference-packages#858. I'm assuming that we'll have to repeat dotnet/source-build-reference-packages#858 for 4.8.0. |
|
No, if we add SBRPs for 4.8.0 we will hit the exact same problem that I'm working around via this PR when consuming GenFacades. |
We already require a VS 17.8 installation so the version bump shouldn't cause any min version requirement issues. This should unblock dotnet/runtime#97087