Skip to content

fix(ci): use backslash paths for msiexec in WinFsp install#236

Merged
FSM1 merged 1 commit into
mainfrom
fix/winfsp-msi-path
Mar 2, 2026
Merged

fix(ci): use backslash paths for msiexec in WinFsp install#236
FSM1 merged 1 commit into
mainfrom
fix/winfsp-msi-path

Conversation

@FSM1

@FSM1 FSM1 commented Mar 2, 2026

Copy link
Copy Markdown
Owner

Summary

  • The WinFsp MSI install step was failing on Windows CI because msiexec.exe treats / as a flag prefix — the forward slashes from github.workspace string concatenation caused "This installation package could not be opened"
  • Use Join-Path for proper Windows backslash paths, quote the path in msiexec arguments, and check the exit code with -PassThru

Test plan

  • Merge and push a staging tag to trigger the deploy-staging workflow
  • Verify the Windows desktop build job passes (WinFsp installs successfully)

🤖 Generated with Claude Code

Summary by CodeRabbit

  • Chores
    • Enhanced deployment process stability with improved Windows file path handling and more robust MSI installation error detection.

msiexec.exe treats '/' as a flag prefix, so the forward slashes from
github.workspace concatenation caused "package could not be opened".
Use Join-Path for proper Windows paths, quote the path in arguments,
and check the msiexec exit code with -PassThru.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Entire-Checkpoint: aad4a654f018
@coderabbitai

coderabbitai Bot commented Mar 2, 2026

Copy link
Copy Markdown

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between 7bbd7bc and c46e07b.

📒 Files selected for processing (1)
  • .github/workflows/deploy-staging.yml

Walkthrough

This pull request modifies the Windows MSI installation workflow step in the deployment script. It replaces hard-coded path strings with PowerShell's Join-Path cmdlet for proper Windows path construction and enhances MSI installation robustness by capturing process handles with PassThru, properly quoting the MSI path, and validating exit codes.

Changes

Cohort / File(s) Summary
GitHub Workflow MSI Installation
.github/workflows/deploy-staging.yml
Replaces hard-coded Windows paths with Join-Path for correct path construction, adds quoted MSI path to msiexec invocation with PassThru process capture, and implements exit code validation for failure detection. Preserves existing hash verification and directory creation logic.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~8 minutes

Possibly related PRs

🚥 Pre-merge checks | ✅ 3
✅ Passed checks (3 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title accurately and concisely describes the main change: fixing CI by using backslash paths for msiexec in WinFsp installation, which is the primary issue addressed in the changeset.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch fix/winfsp-msi-path

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@FSM1 FSM1 merged commit 2d3ec01 into main Mar 2, 2026
21 checks passed
@FSM1 FSM1 deleted the fix/winfsp-msi-path branch March 4, 2026 01:17
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.

1 participant