Skip to content

Conversation

@JunielKatarn
Copy link
Contributor

@JunielKatarn JunielKatarn commented Mar 2, 2022

Description

Set MSBuild property ResolveNuGetPackages to false for projects consuming MSRN.

Type of Change

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

Why

External projects will fail to build due to PackageReference on C++ not being compatible with property ResolveNuGetPackages.

Resolves #9512

What

Sets MSBuild property ResolveNuGetPackages to false. See vnext\PropertySheets\External\Microsoft.ReactNative.Common.props.

Microsoft Reviewers: Open in CodeFlow

@JunielKatarn JunielKatarn requested review from a team as code owners March 2, 2022 00:27
@JunielKatarn JunielKatarn added the PackageReference Modern NuGet restore used in C++ projects label Mar 2, 2022
Copy link

@NikoAri NikoAri left a comment

Choose a reason for hiding this comment

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

:shipit:

@JunielKatarn JunielKatarn merged commit d46f8b3 into microsoft:main Mar 2, 2022
@JunielKatarn JunielKatarn deleted the pkr-dontresolvepkgs-external branch March 2, 2022 19:40
JunielKatarn added a commit to jurocha-ms/react-native-windows that referenced this pull request Mar 2, 2022
* Set ResolveNuGetPackages in external props

* Change files

* Move changes to Microsoft.ReactNative.Common.props
ghost pushed a commit that referenced this pull request Mar 6, 2022
* Set ResolveNuGetPackages in external props (#9628)

* Set ResolveNuGetPackages in external props

* Change files

* Move changes to Microsoft.ReactNative.Common.props

* Change files
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Area: Build Infrastructure PackageReference Modern NuGet restore used in C++ projects

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Errors with Upgrade to 0.68-preview

2 participants