-
Notifications
You must be signed in to change notification settings - Fork 4k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Fix issues with using dogfood compilers #31779
Conversation
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.
Please stop mucking with $(MSBuildAllProjects)
, but otherwise looks good.
<Project> | ||
|
||
<PropertyGroup> | ||
<MSBuildAllProjects>$(MSBuildAllProjects);$(MSBuildThisFileFullPath)</MSBuildAllProjects> |
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.
If/when dev16 is a requirement, you can stop adding this. MSBuild now handles it for you, and adding it just leads to annoying LOH allocations.
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.
yup.
Property _TargetAssemblyProjectName is set by GenerateTemporaryTargetAssembly task. | ||
--> | ||
<PropertyGroup Condition="'$(IsWpfTempProject)' == 'true'"> | ||
<_WpfTempProjectNuGetFileNameNoExt>$(ArtifactsObjDir)$(_TargetAssemblyProjectName)\$(_TargetAssemblyProjectName)$(MSBuildProjectExtension).nuget.g</_WpfTempProjectNuGetFileNameNoExt> |
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.
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.
@jmarolf You can copy the following 2 commits to Arcade SDK:
dotnet/arcade@3d88163
dotnet/arcade@ad721e6
The changes in this PR are Roslyn specific and not needed for Project System.
Run-MSBuild sets various properties based on global variables that were not set in Restore-Project.
Infrastructure only change.