Skip to content

Conversation

@akoeplinger
Copy link
Member

We need to get this msbuild bugfix to resolve a build error during Codespaces prebuild: dotnet/msbuild#8531

We generally try to avoid bumping to a nightly SDK in global.json since it causes pain for contributors so we bump only in the Codespaces prebuild script.

The only downside is that it'll show global.json as modified in git and you need to avoid staging/committing that diff. I think that is an acceptable downside for the next ~1.5 months.

We need to get this msbuild bugfix to resolve a build error during Codespaces prebuild: dotnet/msbuild#8531

We generally avoid bumping to a nightly SDK in global.json since it causes pain for contributors so we bump only in the Codespaces prebuild script.
@ghost
Copy link

ghost commented Mar 13, 2023

Tagging subscribers to this area: @dotnet/runtime-infrastructure
See info in area-owners.md if you want to be subscribed.

Issue Details

We need to get this msbuild bugfix to resolve a build error during Codespaces prebuild: dotnet/msbuild#8531

We generally try to avoid bumping to a nightly SDK in global.json since it causes pain for contributors so we bump only in the Codespaces prebuild script.

The only downside is that it'll show global.json as modified in git and you need to avoid staging/committing that diff. I think that is an acceptable downside for the next ~1.5 months.

Author: akoeplinger
Assignees: -
Labels:

area-Infrastructure

Milestone: -

@akoeplinger akoeplinger merged commit f2a1411 into main Mar 15, 2023
@akoeplinger akoeplinger deleted the fix-codespaces branch March 15, 2023 21:00
akoeplinger added a commit that referenced this pull request Apr 14, 2023
lewing added a commit that referenced this pull request Apr 15, 2023
* Update dependencies from https://github.com/dotnet/arcade build 20230411.8

Microsoft.DotNet.Arcade.Sdk , Microsoft.DotNet.Build.Tasks.Archives , Microsoft.DotNet.Build.Tasks.Feed , Microsoft.DotNet.Build.Tasks.Installers , Microsoft.DotNet.Build.Tasks.Packaging , Microsoft.DotNet.Build.Tasks.TargetFramework , Microsoft.DotNet.Build.Tasks.Templating , Microsoft.DotNet.Build.Tasks.Workloads , Microsoft.DotNet.CodeAnalysis , Microsoft.DotNet.GenAPI , Microsoft.DotNet.GenFacades , Microsoft.DotNet.Helix.Sdk , Microsoft.DotNet.PackageTesting , Microsoft.DotNet.RemoteExecutor , Microsoft.DotNet.SharedFramework.Sdk , Microsoft.DotNet.VersionTools.Tasks , Microsoft.DotNet.XUnitConsoleRunner , Microsoft.DotNet.XUnitExtensions
 From Version 8.0.0-beta.23210.3 -> To Version 8.0.0-beta.23211.8

* Mark NativeBuild_WithDeployOnBuild_UsedByVS as an active issue

* Revert codespaces workaround from #83331

* Bring in missing arcade changes

* Update dependencies from .NET Eng - Latest manually

* Update Version.Details.xml

* Update Versions.props

* Update dependencies from https://github.com/dotnet/arcade build 20230413.1

Microsoft.DotNet.Arcade.Sdk , Microsoft.DotNet.Build.Tasks.Archives , Microsoft.DotNet.Build.Tasks.Feed , Microsoft.DotNet.Build.Tasks.Installers , Microsoft.DotNet.Build.Tasks.Packaging , Microsoft.DotNet.Build.Tasks.TargetFramework , Microsoft.DotNet.Build.Tasks.Templating , Microsoft.DotNet.Build.Tasks.Workloads , Microsoft.DotNet.CodeAnalysis , Microsoft.DotNet.GenAPI , Microsoft.DotNet.GenFacades , Microsoft.DotNet.Helix.Sdk , Microsoft.DotNet.PackageTesting , Microsoft.DotNet.RemoteExecutor , Microsoft.DotNet.SharedFramework.Sdk , Microsoft.DotNet.VersionTools.Tasks , Microsoft.DotNet.XUnitConsoleRunner , Microsoft.DotNet.XUnitExtensions
 From Version 8.0.0-beta.23210.3 -> To Version 8.0.0-beta.23213.1

* Fix ILCompiler build with newer SDK

---------

Co-authored-by: dotnet-maestro[bot] <dotnet-maestro[bot]@users.noreply.github.com>
Co-authored-by: Larry Ewing <[email protected]>
Co-authored-by: Alexander Köplinger <[email protected]>
Co-authored-by: Viktor Hofer <[email protected]>
Co-authored-by: Sven Boemer <[email protected]>
@ghost ghost locked as resolved and limited conversation to collaborators Apr 15, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants