Skip to content

Installer: add prerequisites alias for vcredists#820

Open
johnml1135 wants to merge 1 commit intomainfrom
fix-installer-build
Open

Installer: add prerequisites alias for vcredists#820
johnml1135 wants to merge 1 commit intomainfrom
fix-installer-build

Conversation

@johnml1135
Copy link
Copy Markdown
Contributor

@johnml1135 johnml1135 commented Apr 8, 2026

Summary

  • add a backward-compatible prerequisites package-group alias that forwards to vcredists
  • apply the alias in the tracked redistributables include files used by legacy WiX 3 and WiX 6 installer authoring
  • prevent stale bundle references from breaking installer builds when older authoring still refers to prerequisites

Verification

  • ./Build/Agent/check-and-fix-whitespace.ps1
  • ./Build/Agent/commit-messages.ps1
  • .\build.ps1 -BuildInstaller -Configuration Release

This change is Reviewable

Copilot AI review requested due to automatic review settings April 8, 2026 01:47
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Note

Copilot was unable to run its full agentic suite in this review.

Adds a backward-compatible WiX PackageGroup alias named prerequisites that forwards to the existing vcredists group, preventing legacy bundle authoring from breaking when it still references prerequisites.

Changes:

  • Introduce PackageGroup Id="prerequisites" as an alias to vcredists
  • Apply the alias across the tracked redistributables include files used by WiX 3 and WiX 6 authoring

Reviewed changes

Copilot reviewed 3 out of 3 changed files in this pull request and generated no comments.

File Description
FLExInstaller/wix6/Shared/Common/Redistributables.wxi Adds prerequisites package-group alias forwarding to vcredists for shared WiX 6 authoring.
FLExInstaller/wix6/Redistributables.wxi Adds prerequisites alias for the WiX 6 redistributables include.
FLExInstaller/Redistributables.wxi Adds prerequisites alias for the legacy/alternate redistributables include.

@github-actions
Copy link
Copy Markdown

github-actions bot commented Apr 8, 2026

NUnit Tests

    1 files  ±0      1 suites  ±0   6m 15s ⏱️ +27s
4 103 tests ±0  4 032 ✅ ±0  71 💤 ±0  0 ❌ ±0 
4 112 runs  ±0  4 041 ✅ ±0  71 💤 ±0  0 ❌ ±0 

Results for commit c8dad5c. ± Comparison against base commit 7cb3fe9.

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