Skip to content

fix: add rpath for Homebrew lib directory to resolve libsteam_api#24

Merged
unhappychoice merged 1 commit intomainfrom
fix/add-libsteam-api-rpath
Feb 13, 2026
Merged

fix: add rpath for Homebrew lib directory to resolve libsteam_api#24
unhappychoice merged 1 commit intomainfrom
fix/add-libsteam-api-rpath

Conversation

@unhappychoice
Copy link
Owner

@unhappychoice unhappychoice commented Feb 13, 2026

Summary

  • Add @executable_path/../lib/steamfetch rpath so the binary can find libsteam_api.dylib when installed via Homebrew (bin/ -> lib/steamfetch/)
  • Existing @executable_path / $ORIGIN rpath is preserved for install.sh and local builds
  • Companion PR: https://github.com/unhappychoice/homebrew-tap — installs libsteam_api to lib/steamfetch/

Fixes: dyld: Library not loaded: @loader_path/libsteam_api.dylib on macOS when installed via Homebrew

Summary by CodeRabbit

  • Chores
    • Improved build configuration for Linux and macOS to enhance how the application discovers and locates Steam API dependencies at runtime. This update enables more flexible deployment options, supports a broader range of installation configurations, and significantly improves application portability across diverse system environments and distributions while maintaining backward compatibility.

@coderabbitai
Copy link

coderabbitai bot commented Feb 13, 2026

📝 Walkthrough

Walkthrough

The build.rs script now adds an additional rpath configuration for Linux and macOS targets, enabling the binary to locate Steam API libraries at ../lib/steamfetch relative to the executable. This supplements the existing rpath entry, providing an alternate search path for runtime library loading.

Changes

Cohort / File(s) Summary
Build Configuration
build.rs
Added secondary rpath directive for Linux and macOS builds pointing to ../lib/steamfetch directory to enable library resolution at alternate runtime location.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 minutes

Poem

🐰 A rabbit builds with careful care,
Paths added here and paths added there!
Steam libraries now have a place to hide,
In ../lib/steamfetch, nested inside.
The binary hops and finds its way,
With rpath magic, hip-hip-hooray! 🏃✨

🚥 Pre-merge checks | ✅ 2 | ❌ 2
❌ Failed checks (2 warnings)
Check name Status Explanation Resolution
Docstring Coverage ⚠️ Warning Docstring coverage is 0.00% which is insufficient. The required threshold is 80.00%. Write docstrings for the functions missing them to satisfy the coverage threshold.
Merge Conflict Detection ⚠️ Warning ⚠️ Unable to check for merge conflicts: Failed to fetch base branch: From https://github.com/unhappychoice/steamfetch
! [rejected] main -> main (non-fast-forward)
+ d279d8e...6e9a7a7 main -> origin/main (forced update)
✅ Passed checks (2 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: adding an rpath for resolving libsteam_api library dependencies, which directly relates to the build.rs modifications.

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

✨ Finishing touches
  • 📝 Generate docstrings
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch fix/add-libsteam-api-rpath
⚔️ Resolve merge conflicts (beta)
  • Auto-commit resolved conflicts to branch fix/add-libsteam-api-rpath
  • Create stacked PR with resolved conflicts
  • Post resolved changes as copyable diffs in a comment

No actionable comments were generated in the recent review. 🎉


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.

@unhappychoice unhappychoice merged commit 6e9a7a7 into main Feb 13, 2026
7 of 8 checks passed
@unhappychoice unhappychoice deleted the fix/add-libsteam-api-rpath branch February 13, 2026 12:11
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