Skip to content

fix(ci): symlink FUSE-T pkgconfig as fuse.pc for macOS builds#225

Merged
FSM1 merged 1 commit into
mainfrom
fix/macos-fuse-t-pkgconfig
Mar 1, 2026
Merged

fix(ci): symlink FUSE-T pkgconfig as fuse.pc for macOS builds#225
FSM1 merged 1 commit into
mainfrom
fix/macos-fuse-t-pkgconfig

Conversation

@FSM1

@FSM1 FSM1 commented Mar 1, 2026

Copy link
Copy Markdown
Owner

Summary

  • FUSE-T installs fuse-t.pc at /Library/Application Support/fuse-t/pkgconfig/, but the vendored fuser build.rs probes for fuse or osxfuse via pkg-config
  • On CI runners (which only have FUSE-T, not macFUSE), no fuse.pc exists, causing cargo build to fail
  • Fix: after brew install --cask fuse-t, symlink fuse-t.pc/usr/local/lib/pkgconfig/fuse.pc
  • Applied to all 4 workflows that install FUSE-T: ci.yml (×2), e2e-desktop.yml, deploy-staging.yml

Test plan

  • Verify macOS Cargo Check passes on this PR
  • After merge, verify Build macOS Desktop succeeds on push to main

🤖 Generated with Claude Code

Summary by CodeRabbit

  • Chores
    • Updated macOS build and testing workflows with enhanced FUSE-T library configuration to improve consistency across continuous integration, staging deployment, and end-to-end testing environments.

FUSE-T installs its pkgconfig file as fuse-t.pc in a non-standard
location (/Library/Application Support/fuse-t/pkgconfig/), but the
vendored fuser build.rs probes for "fuse" or "osxfuse". On CI runners
without macFUSE, there is no fuse.pc file, causing the build to fail.

Fix: after installing FUSE-T, create a symlink from fuse-t.pc to
/usr/local/lib/pkgconfig/fuse.pc so pkg-config finds it under the
expected name. Applied to all 4 workflow files that install FUSE-T.

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

coderabbitai Bot commented Mar 1, 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 c8329c6 and 15011b1.

📒 Files selected for processing (3)
  • .github/workflows/ci.yml
  • .github/workflows/deploy-staging.yml
  • .github/workflows/e2e-desktop.yml

Walkthrough

Three GitHub Actions workflow files are modified to enhance macOS FUSE-T installation. Each change replaces a single brew install command with a multi-line script that installs FUSE-T, creates /usr/local/lib/pkgconfig, and symlinks fuse-t.pc for pkg-config discovery.

Changes

Cohort / File(s) Summary
macOS FUSE-T Setup
.github/workflows/ci.yml, .github/workflows/deploy-staging.yml, .github/workflows/e2e-desktop.yml
Replaced single-line brew install --cask fuse-t with multi-line script that creates /usr/local/lib/pkgconfig directory and establishes symlink from fuse-t.pc to /usr/local/lib/pkgconfig/fuse.pc for pkg-config visibility.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~12 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 describes the main change: symlinking FUSE-T's pkgconfig file as fuse.pc for macOS CI builds to resolve build failures.
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/macos-fuse-t-pkgconfig

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 f2cfb0f into main Mar 1, 2026
16 checks passed
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