Skip to content

[release/v7.5] Fix R2R for fxdependent packaging #26797

Merged
daxian-dbw merged 1 commit intoPowerShell:release/v7.5from
daxian-dbw:backport/release/v7.5/26131-51a21999d
Feb 12, 2026
Merged

[release/v7.5] Fix R2R for fxdependent packaging #26797
daxian-dbw merged 1 commit intoPowerShell:release/v7.5from
daxian-dbw:backport/release/v7.5/26131-51a21999d

Conversation

@daxian-dbw
Copy link
Member

Backport of #26131 to release/v7.5

Triggered by @daxian-dbw on behalf of @adityapatwardhan

Original CL Label: CL-BuildPackaging

/cc @PowerShell/powershell-maintainers

Impact

REQUIRED: Choose either Tooling Impact or Customer Impact (or both). At least one checkbox must be selected.

Tooling Impact

  • Required tooling change
  • Optional tooling change (include reasoning)

This fix is required for correct ReadyToRun compilation settings in FxDependent packaging builds. Without this fix, ReadyToRun may be incorrectly enabled or disabled for specific deployment types.

Customer Impact

  • Customer reported
  • Found internally

Regression

REQUIRED: Check exactly one box.

  • Yes
  • No

This is not a regression.

Testing

Manual testing of build process for FxDependent and FxDependentDeployment scenarios to verify ReadyToRun settings work correctly for different deployment types.

Risk

REQUIRED: Check exactly one box.

  • High
  • Medium
  • Low

This is a build configuration change that only affects ReadyToRun settings for different deployment types. It fixes an issue introduced by PR #25837 and has been tested in the master branch.

@daxian-dbw daxian-dbw requested a review from a team as a code owner February 12, 2026 22:06
@daxian-dbw daxian-dbw added the CL-BuildPackaging Indicates that a PR should be marked as a build or packaging change in the Change Log label Feb 12, 2026
Copilot AI review requested due to automatic review settings February 12, 2026 22:06
@daxian-dbw daxian-dbw added the CL-BuildPackaging Indicates that a PR should be marked as a build or packaging change in the Change Log label Feb 12, 2026
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Backport to release/v7.5 that fixes ReadyToRun (R2R) settings for framework-dependent packaging by making PublishReadyToRun / PublishReadyToRunEmitSymbols conditional on the deployment type (AppDeployment) rather than unconditionally enabling them for all Release builds.

Changes:

  • Disable R2R for FxDependent deployments.
  • Enable R2R (and symbol emission) for FxDependentDeployment and SelfContained deployments.
  • Remove unconditional R2R enabling from the Release configuration property group.

@daxian-dbw daxian-dbw merged commit e0b3e8f into PowerShell:release/v7.5 Feb 12, 2026
41 of 43 checks passed
@daxian-dbw daxian-dbw deleted the backport/release/v7.5/26131-51a21999d branch February 12, 2026 22:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

CL-BuildPackaging Indicates that a PR should be marked as a build or packaging change in the Change Log

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants