- 
                Notifications
    You must be signed in to change notification settings 
- Fork 6
Bump to .NET 6 stable #32
Conversation
| I'm HTTP 401 to access the CI build logs, but my guess is the samples don't build because of #33, or did I mess up somewhere else? | 
| There is some failure in the iOS workload: We're using Xamarin.Legacy.Sdk + .NET 6 GA, in AndroidX and Google Play Services bindings... Maybe there is something I can do to work around the iOS issue. | 
* Install .NET 6 GA * Xcode 13.1 is default now: actions/runner-images#4355
$(SupportedOSPlatformVersion) should be 10.14
Fixes:
    D:\a\1\s\samples\Hello\Hello.csproj error MSB4057: The target "_GetRestoreSettingsPerFramework" does not exist in the project.
            
          
                src/Xamarin.Legacy.Sdk/Sdk/Sdk.props
              
                Outdated
          
        
      | <!-- | ||
| HACK: declare a _GetRestoreSettingsPerFramework target that some versions of NuGet expect to exist. | ||
| This target should get replaced by NuGet.targets if needed: | ||
| https://github.com/NuGet/NuGet.Client/blob/914cacea77660b8758014912ee127c82469ac1d9/src/NuGet.Core/NuGet.Build.Tasks/NuGet.targets#L635-L651 | ||
| --> | ||
| <Target Name="_GetRestoreSettingsPerFramework" Returns="@(_RestoreSettingsPerFramework)"> | ||
| <ItemGroup> | ||
| <_RestoreSettingsPerFramework Include="$([System.Guid]::NewGuid())"> | ||
| <RestoreAdditionalProjectSources>$(RestoreAdditionalProjectSources)</RestoreAdditionalProjectSources> | ||
| <RestoreAdditionalProjectFallbackFolders>$(RestoreAdditionalProjectFallbackFolders)</RestoreAdditionalProjectFallbackFolders> | ||
| <RestoreAdditionalProjectFallbackFoldersExcludes>$(RestoreAdditionalProjectFallbackFoldersExcludes)</RestoreAdditionalProjectFallbackFoldersExcludes> | ||
| </_RestoreSettingsPerFramework> | ||
| </ItemGroup> | ||
| </Target> | 
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@mfkl adding this solved one issue, were you getting errors about this target missing?
After adding this, I hit a new missing target, which is too large to replicate in this project:
I think these were very late changes to improve no-op restore performance in NuGet in .NET 6.
Need to think on this one...
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Ok remembering now... This needs to be updated for VS 2022:
| <VsInstallRoot Condition=" '$(VsInstallRoot)' == '' and Exists ('$(MSBuildProgramFiles32)/Microsoft Visual Studio/2019/Preview/') ">$(MSBuildProgramFiles32)/Microsoft Visual Studio/2019/Preview/</VsInstallRoot> | |
| <VsInstallRoot Condition=" '$(VsInstallRoot)' == '' ">$(MSBuildProgramFiles32)/Microsoft Visual Studio/2019/Enterprise/</VsInstallRoot> | 
This reverts commit 81dbd0d.
| This is now hitting a new iOS error:  | 
This reverts commit 8176afc.
This solves an assembly-load error at build time.
| Good job! | 
No description provided.