Skip to content
This repository has been archived by the owner on Dec 20, 2024. It is now read-only.

Build: .NET 6 SDK, drop ilrepack from build, change some target frameworks #19

Draft
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

sandyarmstrong
Copy link
Member

.NET Core 2.1 is not supported on arm64 Macs. .NET 6.0.300 SDK is a
long-term support release with broad compatibility.

Newer Microsoft.Build package versions no longer have netstandard2.0
support (see dotnet/msbuild#4521). Update
Xamarin.MSBuild.Sdk to multi-target net6.0 and net472 to address
this.

Xamarin.BuildConsolidator depends on ILRepack, which is not supported
in .NET 6. The only actively-used consolidation support comes from
PrepareConsolidationProject in Xamarin.MSBuild.Sdk, so remove the
unused projects from the build.

…works

.NET Core 2.1 is not supported on arm64 Macs. .NET 6.0.300 SDK is a
long-term support release with broad compatibility.

Newer `Microsoft.Build` package versions no longer have `netstandard2.0`
support (see dotnet/msbuild#4521). Update
`Xamarin.MSBuild.Sdk` to multi-target `net6.0` and `net472` to address
this.

`Xamarin.BuildConsolidator` depends on ILRepack, which is not supported
in .NET 6. The only actively-used consolidation support comes from
`PrepareConsolidationProject` in `Xamarin.MSBuild.Sdk`, so remove the
unused projects from the build.
@sandyarmstrong sandyarmstrong marked this pull request as draft July 5, 2022 16:51
@sandyarmstrong
Copy link
Member Author

Converting to draft until unit tests are fixed.

@KirillOsenkov
Copy link
Member

FYI I fixed ILRepack and it's glorious now :)

@sandyarmstrong
Copy link
Member Author

FYI I fixed ILRepack and it's glorious now :)

@KirillOsenkov that is awesome! Sadly mirepoix isn't really being used anywhere anymore, but there are TONS of projects using ilrepack so glad to hear it's working again. 😃

@KirillOsenkov
Copy link
Member

Yeah I realize I'm a bit late with this, but hey! :)

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants