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

Wax error for Guid value #67

Open
TimBakkerTooling opened this issue Oct 5, 2020 · 6 comments
Open

Wax error for Guid value #67

TimBakkerTooling opened this issue Oct 5, 2020 · 6 comments

Comments

@TimBakkerTooling
Copy link

When creating the Wix project file with Wax Setup editor i get the following compilation error:

The Component/@Guid attribute's value, '1c3291b8-f364-4d4b-beb0-26f8fa743e78', is a mixed-case guid. All letters in a guid value should be uppercase.

@tom-englert
Copy link
Owner

I never saw such an Error. Maybe you can turn off this error?

@TimBakkerTooling
Copy link
Author

Yes i found out that the error message level was "Pedantic". Setting back to normal fixed the issue.
However, i fixed the error by adding .ToUpper() at the file creating method.

@tom-englert
Copy link
Owner

Can you provide a PR?

@TimBakkerTooling
Copy link
Author

I just created a Pull Request! I'm not sure what the correct versionnumbering is.

Also fixed another error about KeyPaths

@tom-englert
Copy link
Owner

The KeyPath is not really required here, but having it explicit is also fine.

@Miles-Davies-HORIBA
Copy link

@tom-englert the reason for the case needing to be upper case is detailed here

Note that the letters of these GUIDs must be uppercase. Utilities such as GUIDGEN can generate GUIDs containing lowercase letters. The lowercase letters must be changed to uppercase to make these valid component code GUIDs.

While it is OK to not have the KeyPath flag set it is more correct to do so. Especially when there is one file per component.

If the value is not null, then KeyPath is either a primary key into the Registry, ODBCDataSource, or File tables depending upon the Attribute value. If KeyPath is null, then the folder of the Directory_ column is used as the key path.

For file components this should probably be set unless the file is protected by WRP (not commonly the case where Wax is used).

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

No branches or pull requests

3 participants