Skip to content

Ensure that reference conflicts are resolved in favor of the shared assemblies #5083

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
wants to merge 1 commit into
base: release/9.0
Choose a base branch
from

Conversation

Tanya-Solyanik
Copy link
Contributor

@Tanya-Solyanik Tanya-Solyanik commented May 13, 2025

issue: #4991

Customer impact:
Assemblies that are present in the shared location are unexpectedly copied to the app local.

Workaround:
remove extra assemblies post build or use ExcludeAssets property with the PackageReference tag.

Regression?
perceived as a regression, it depends on whether the package referenced explicitly in the project file is newer or not that the GA version of package from the shared location.

Details
Desktop package was not updating the PlatformManifest file with every servicing release. Instead, this file was checked in and versions were pinned to GA. We change this logic to keep them up-to-date with the latest servicing release. This is the default arcade behavior.

Verification:
Compare the platform manifest files.
In the new package manifest file all except the following assembly versions were updated:
D3DCompiler_47_cor3.dll|Microsoft.WindowsDesktop.App.Ref||10.0.22621.3233
PresentationNative_cor3.dll|Microsoft.WindowsDesktop.App.Ref||9.0.24.47404
System.Diagnostics.EventLog.Messages.dll|Microsoft.WindowsDesktop.App.Ref|9.0.0.0|0.0.0.0

…ith the latest servicing release. THis is the default arcade behavior, thus let it generate the manifest.
@ViktorHofer
Copy link
Member

This probably needs an ask mode template and Tactics approval.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants