Skip to content

fix installer lib not loading in ide#643

Merged
planetchili merged 1 commit into
mainfrom
fix/wix-proj-ide
Jun 9, 2026
Merged

fix installer lib not loading in ide#643
planetchili merged 1 commit into
mainfrom
fix/wix-proj-ide

Conversation

@planetchili

Copy link
Copy Markdown
Collaborator

fix issue where wix library project gets/stays unloaded in IDE view

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull request overview

Adjusts the WiX library project so it stays loadable in Visual Studio by avoiding references to generated authoring under intermediate output paths and by setting sensible default MSBuild properties.

Changes:

  • Added default Configuration/Platform values to the WiX library project to stabilize design-time evaluation.
  • Moved the generated UciDist.wxs fragment output under the project directory (Generated\) and updated build/compile inputs accordingly.
  • Added .gitignore entries for the new generated output folder and an additional UCI working directory name.

Reviewed changes

Copilot reviewed 2 out of 3 changed files in this pull request and generated 1 comment.

File Description
IntelPresentMon/PMInstallerLib/PMInstallerLib.wixproj Generates/consumes the harvested UCI fragment from a stable project-relative path to improve IDE load behavior.
IntelPresentMon/ControlLib/uci/.gitignore Ignores an additional UCI directory (external-off/).
.gitignore Ignores PMInstallerLib/Generated/ to prevent committing harvested WiX authoring.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread IntelPresentMon/PMInstallerLib/PMInstallerLib.wixproj
@planetchili planetchili merged commit 90a4629 into main Jun 9, 2026
1 check passed
@planetchili planetchili deleted the fix/wix-proj-ide branch June 9, 2026 17:21
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.

2 participants