Skip to content
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

Patch applicability does not check all installed product for non-specific patches #4366

Closed
wixbot opened this issue Mar 28, 2014 · 3 comments
Assignees
Milestone

Comments

@wixbot
Copy link

wixbot commented Mar 28, 2014

The original optimization in patch applicability in mspengine.cpp was to only check all installed products if even 1 patch does not target a specific ProductCode. If all patches target a specific ProductCode, we only check that list.

However, Burn determines which to do based on whether or not any TargetProductCode elements are in the manifest regardless of whether any of the patches are non-specific.

Rather than having Burn check whether any MsiPatch/TargetProduct/TargetProductCode elements are non-specific via XPath, we should do that in the binder. The actual error is in ChainPackageInfo.cs where we check that element. If found, the assumption is that the patch targets a specific ProductCode. however, you also need to check if the Validate attribute is "true" (or maybe there's multiple versions of the XML blob where either is possible, in which case check the latter first then the former). The same assumption is there for the UpgradeCode as well. All the child elements of MsiPatch/TargetProduct have that schema, FWIW.

Originally opened by heaths

@wixbot
Copy link
Author

wixbot commented Apr 3, 2014

Area changed from linker to burn
Release changed from v3.8 to v3.x

@wixbot
Copy link
Author

wixbot commented Jun 14, 2014

Originally changed by heaths
AssignedTo set to heaths

@wixbot
Copy link
Author

wixbot commented Jun 15, 2014

Originally changed by heaths
Resolution set to fixed
Status changed from Open to Resolved

@wixbot wixbot added this to the v3.x milestone Dec 20, 2015
@wixbot wixbot closed this as completed Dec 20, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants