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

Support WiX3-compantible auto-guids for registry key path #589

Closed
wants to merge 1 commit into from

Conversation

nirbar
Copy link
Contributor

@nirbar nirbar commented Jan 14, 2025

Fix #8896
By default, generated GUID will be compatible with WiX4+
Added an attribute to generate WiX3-compatible GUID: Component/@WiX3CompatibleGuid=yes/no


Assert.Equal(2, results.Length);
Assert.Contains(results, component => component.Contains("WiX3CompatibleGuid") && component.Contains("8BAF5399-2FD2-50B6-ABDA-98FB3A5BB148", StringComparison.InvariantCultureIgnoreCase));
Assert.Contains(results, component => component.Contains("WiX4CompatibleGuid") && !component.Contains("8BAF5399-2FD2-50B6-ABDA-98FB3A5BB148", StringComparison.InvariantCultureIgnoreCase));
Copy link
Contributor

Choose a reason for hiding this comment

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

I wonder if this might be 'better' as a strict equivalency test to the Wix4CompatibleGuid... in this manner the test would both verify that it's not the same as Wix3Guid, but also that if a future (potentially inadvertant) change makes things different from Wix4Guid it would be detected by this test.

This comment might be moot if there are already Wix4Guid tests elsewhere (I haven't looked specifically), and arguably having a dedicated set of guid tests is probably desirable elsewhere also. But something for right now would be good..

Copy link
Contributor Author

Choose a reason for hiding this comment

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

@bevanweiss I added a test to check compatibility with WiX v4.0.0 auto-genertated GUIDs for registry values under HKLM, HKU, HKCR, and for a file under ProgramFilesFolder

@robmen robmen self-assigned this Feb 11, 2025
@robmen robmen closed this in #600 Feb 12, 2025
@github-actions github-actions bot locked and limited conversation to collaborators Feb 12, 2025
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Auto-generated component GUID for registry is incompatible with WiX3
3 participants