Change Test-DotnetNewTemplates' nugetCacheRoot to C:\#6591
Closed
lauren-ciha wants to merge 3 commits into
Closed
Conversation
Member
Author
|
/azp run |
|
Azure Pipelines successfully started running 2 pipeline(s). |
Member
Author
|
Closing this PR because we chose another solution |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Resolves the recent pipeline errors in #6583.
In
Test-DotnetNewTemplates.ps1, a temporary NuGet cache is created to keep test template dependencies separate from the user's NuGet configuration. Originally, this path was set within the repo's path. When files were added to this path, it exceeded the Windows MAX_PATH of 260 characters. The WinUI Xaml compiler resolves transitive dependencies using APIs that are not long-path aware. Therefore, when the dependency versions were longer and the nuget cache path was deep, the total path length was longer than the maximum path length, causing the XAML compiler to not emit App.g.i.cs. When the templates were built, the missing App.g.i.cs caused a CS2001 build error. We can prevent this build error by setting the test NuGet cache path to a shorter string. Currently, this is set to$env:TEMP, but this is not the only path option.$env:TEMPwas chosen because it's a short path name that is already a location for temporary files like the test cache.A microsoft employee must use /azp run to validate using the pipelines below.
WARNING:
Comments made by azure-pipelines bot maybe inaccurate.
Please see pipeline link to verify that the build is being ran.
For status checks on the main branch, please use TransportPackage-Foundation-PR
(https://microsoft.visualstudio.com/ProjectReunion/_build?definitionId=81063&_a=summary)
and run the build against your PR branch with the default parameters.