Skip to content

fix: %(AdditionalDependencies) metadata typo in cpp-app/cpp-lib templates and package projects#15720

Merged
acoates-ms merged 2 commits intomicrosoft:mainfrom
PHILLIPS71:fix/msbuild-additional-dependencies-typo
Mar 4, 2026
Merged

fix: %(AdditionalDependencies) metadata typo in cpp-app/cpp-lib templates and package projects#15720
acoates-ms merged 2 commits intomicrosoft:mainfrom
PHILLIPS71:fix/msbuild-additional-dependencies-typo

Conversation

@PHILLIPS71
Copy link
Contributor

@PHILLIPS71 PHILLIPS71 commented Mar 3, 2026

Description

Type of Change

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

Why

The <AdditionalDependencies> linker property in the cpp-app and cpp-lib templates (and all derived package projects) contained a misspelling: %(AdditionalDependenices) instead of %(AdditionalDependencies).

MSBuild silently resolves unknown item metadata references to an empty string with no warning or error, so any additional link dependencies inherited from parent property sheets were being silently dropped at link time.

What

Corrected the metadata reference from %(AdditionalDependenices) to %(AdditionalDependencies) in the <Link> item group of all affected .vcxproj files.

Screenshots

N/A

Testing

N/A

Changelog

Should this change be included in the release notes: yes

Fixed a silent MSBuild linker bug in the cpp-app and cpp-lib templates where a typo in %(AdditionalDependencies) caused inherited link dependencies to be silently dropped. Projects already scaffolded from these templates should apply the same correction to their .vcxproj files.

Microsoft Reviewers: Open in CodeFlow

@PHILLIPS71 PHILLIPS71 requested a review from a team as a code owner March 3, 2026 23:21
@PHILLIPS71
Copy link
Contributor Author

@microsoft-github-policy-service agree

@acoates-ms
Copy link
Contributor

/azp run

@azure-pipelines
Copy link

Azure Pipelines successfully started running 2 pipeline(s).

@acoates-ms
Copy link
Contributor

You 'll need to run yarn change in the root of the repo to generate a change file.

@PHILLIPS71
Copy link
Contributor Author

You 'll need to run yarn change in the root of the repo to generate a change file.

Cheers, I've added one.

@acoates-ms
Copy link
Contributor

/azp run

@azure-pipelines
Copy link

Azure Pipelines successfully started running 2 pipeline(s).

@acoates-ms acoates-ms merged commit b67851f into microsoft:main Mar 4, 2026
46 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants