Fix build to only enable ready-to-run for the Release configuration#26290
Fix build to only enable ready-to-run for the Release configuration#26290daxian-dbw merged 2 commits intoPowerShell:masterfrom
Conversation
There was a problem hiding this comment.
Pull Request Overview
This PR fixes the build configuration to enable ready-to-run compilation only for Release builds, preventing debugging issues in Debug builds. Previously, ready-to-run was enabled for all configurations when AppDeployment was set to SelfContained, which made it impossible to debug the code in Debug configuration.
Key changes:
- Split the
AppDeploymentproperty group to separate ready-to-run settings from other self-contained deployment settings - Added a configuration condition to enable ready-to-run only when both
AppDeploymentisSelfContainedandConfigurationisRelease
Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.
daxian-dbw
added
the
CL-BuildPackaging
|
marked for backport because I'm not sure how far back this bug was introduced. |
|
📣 Hey @@daxian-dbw, how did we do? We would love to hear your feedback with the link below! 🗣️ 🔗 https://aka.ms/PSRepoFeedback |
|
Not applicable to 7.4 |
daxian-dbw
added
Backport-7.5.x-Migrated
Backport-7.5.x-Consider
and removed
Backport-7.5.x-Consider
Backport-7.5.x-Migrated
labels
daxian-dbw
added
Backport-7.4.x-Done
Backport-7.5.x-Done
and removed
Backport-7.4.x-Migrated
Backport-7.5.x-Migrated
labels
PR Summary
Fix build to only enable ready-to-run for the Release configuration.
Currently,
Start-PSBuildwill enable ready-to-run even for 'Debug' configuration, which makes it impossible to debug the code. This PR fixes it to only enable ready-to-run for Release configuration.