Skip to content

[0.81] Fix Circular Dependency issues for New Arch Apps#15805

Merged
vineethkuttan merged 1 commit intomicrosoft:0.81-stablefrom
vineethkuttan:circularfix
Mar 19, 2026
Merged

[0.81] Fix Circular Dependency issues for New Arch Apps#15805
vineethkuttan merged 1 commit intomicrosoft:0.81-stablefrom
vineethkuttan:circularfix

Conversation

@vineethkuttan
Copy link
Contributor

@vineethkuttan vineethkuttan commented Mar 18, 2026

Description

Fixing Circular Dependency issues for New Arch Apps

Type of Change

  • Bug fix (non-breaking change which fixes an issue)

Why

Due to ongoing issues in microsoft/cppwinrt#950 , we added a temporary fix in #13338 to make the projects build with -m , i.e one concurrent process for building the RNW App, refer here

What

Inspired by this changes microsoft/winget-cli#5932 , I have defined CppWinRTGenerateWindowsMetadata to prevent the circular reference in calculating the value

Testing

Tested in playground

Changelog

Should this change be included in the release notes: yes

Circular dependency build issues for RNW Fabric App has been fixed

Microsoft Reviewers: Open in CodeFlow

* Try fixing microsoft#13599

* Change files

* fix by looking in winget-cli

- microsoft/winget-cli#5932

* Make changes in MyApp and revert changes in the prop

* Add Force Synchronous PDB Write

* Remove the direct Build.0/Deploy.0 entries for the vcxproj

* fix folly build issues

- reverting playground.yml

* Set WinMD generation to true
@vineethkuttan vineethkuttan requested review from a team as code owners March 18, 2026 04:09
@vineethkuttan vineethkuttan changed the base branch from main to 0.81-stable March 18, 2026 04:09
@vineethkuttan vineethkuttan removed request for a team March 18, 2026 04:11
@vineethkuttan
Copy link
Contributor Author

/azp run

@azure-pipelines
Copy link
Contributor

Azure Pipelines successfully started running 2 pipeline(s).

@vineethkuttan vineethkuttan added the AutoMerge Causes a PR to be automatically merged once all requirements are passed (label drives bot activity) label Mar 19, 2026
Copy link
Contributor

@protikbiswas100 protikbiswas100 left a comment

Choose a reason for hiding this comment

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

LGTM

@vineethkuttan vineethkuttan merged commit f55ee8d into microsoft:0.81-stable Mar 19, 2026
110 checks passed
@vineethkuttan vineethkuttan deleted the circularfix branch March 19, 2026 06:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

AutoMerge Causes a PR to be automatically merged once all requirements are passed (label drives bot activity)

Projects

None yet

Development

Successfully merging this pull request may close these issues.

New arch app first builds sometimes fail with circular dependency error MSB4006

2 participants