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

FileSearch does not work in a Merge Module #8558

Open
1 task done
MatthiasRInst opened this issue Jun 10, 2024 · 0 comments · May be fixed by wixtoolset/wix#536
Open
1 task done

FileSearch does not work in a Merge Module #8558

MatthiasRInst opened this issue Jun 10, 2024 · 0 comments · May be fixed by wixtoolset/wix#536
Assignees
Labels

Comments

@MatthiasRInst
Copy link

WiX Version

4.0.5

.NET or MSBuild or Visual Studio Version

Visual Studio 17.9.7

HeatWave Version

1.0.4.5

Windows Version

Win10 22H2

Repro Repo

No response

Repro Steps

Build a Merge Module with the following WiX source code:

<Module Id="FileSearch" Language="1033" Version="1.0.0.0"
          Guid="{110CF125-7A17-41F1-9C16-DDDAE0A1B03A}" InstallerVersion="500">
		<SummaryInformation Manufacturer="Licence Owner" />
    <Property Id="FILEFOUND" Secure="yes">
      <DirectorySearch Id="Windows" Path="[WindowsFolder]" >
        <FileSearch Id="MyFile_ini" Name="MyFile.ini" />
      </DirectorySearch>
    </Property>
</Module>

Actual Result

Error message:

WIX0001 System.InvalidOperationException: The value '' is not a legal identifier and therefore cannot be modularized. C:\MySourceCodePath\Installation\MergeModules\FileSearch\wix.exe 1

Expected Result

I expect the code to compile, resulting in a Merge Module with entries in the AppSearch, Signature, and DrLocator tables.

In WiX 3, this code compiled.

Acknowledgements

  • I acknowledge that this is a fully completed bug report. It is not a question or attempt to get help debugging my issue (because those should be sent to Discussions).
@barnson barnson self-assigned this Jun 18, 2024
@barnson barnson added binder and removed triage labels Jun 18, 2024
barnson added a commit to wixtoolset/wix that referenced this issue Jun 30, 2024
@robmen robmen linked a pull request Jul 12, 2024 that will close this issue
barnson added a commit to wixtoolset/wix that referenced this issue Jul 15, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants