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

Ensure naked files take Subdirectory into account. #557

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

barnson
Copy link
Member

@barnson barnson commented Aug 3, 2024

Naked files generated their ids using the attributes that are common to both naked and clothed files. But naked files also support Subdirectory to magic up a subdirectory in a specified directory (Directory) or the default INSTALLFOLDER. That subdirectory needs to factor in to the generated file id (which is then used as the component id too). Without it, generated ids for files with the same name but from different Subdirectory values would be duplicated. (Authored file ids must also continue to be supported.)

Naked files now generate different file and component ids. :(

Fixes wixtoolset/issues#8674

@barnson barnson requested a review from robmen August 3, 2024 20:38
Naked files generated their ids using the attributes that are common to
both naked and clothed files. But naked files also support @subdirectory
to magic up a subdirectory in a specified directory (@Directory) or the
default INSTALLFOLDER. That subdirectory needs to factor in to the
generated file id (which is then used as the component id too).
Without it, generated ids for files with the same name but from
different @subdirectory values would be duplicated. (Authored file ids
must also continue to be supported.)

Naked files now generate different file and component ids. :(

Fixes wixtoolset/issues#8674
@barnson barnson self-assigned this Sep 3, 2024
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.

Naked Files can result in WIX0095 errors in mysterious circumstances
1 participant