Repository navigation
Add PublishReadyToRun to DotNetRestoreSettings #4109
Description
Discussed in https://github.com/orgs/cake-build/discussions/4108
Originally posted by Trisped January 19, 2023
According to https://learn.microsoft.com/en-us/dotnet/core/compatibility/sdk/6.0/publish-readytorun-requires-restore-change#recommended-action :
In .NET 6, dotnet restore followed by dotnet publish -p:PublishReadyToRun=true --no-restore will fail with the NETSDK1095 error. This is because the crossgen binary is now shipped as a separate NuGet package, and so needs to be part of the restore operation for publishing to succeed.
According to https://cakebuild.net/api/Cake.Common.Tools.DotNet.Restore/DotNetRestoreSettings/ , there is not a PublishReadyToRun property like there is in DotNetPublishSettings.
There is a work around (set NoRestore to false), but it requires restoring twice, if you have already restored (for example, so you could test before publishing).
Activity
SARAVANA1501 commentedon Mar 25, 2023
Is it up for contribution?
augustoproiete commentedon Mar 27, 2023
Hey @SARAVANA1501, no this one already has a pull-request that is waiting to be merged - #4110
augustoproiete commentedon Mar 27, 2023
Assigned to @Marusyk
(cake-buildGH-4109) Add PublishReadyToRun to DotNetRestoreSettings
(cake-buildGH-4109) Add PublishReadyToRun to DotNetRestoreSettings
Merge pull request #4110 from Marusyk/rmarusyk/4109
cake-build-bot commentedon Jul 9, 2023
🎉 This issue has been resolved in version v3.1.0 🎉
The release is available on:
Your GitReleaseManager bot 📦🚀