feat: Linux compatibility fixes for cross-platform PAI #288
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Fixes 4 critical Linux compatibility issues, enabling PAI installation
on Google Gemini CLI and other Linux environments.
What's fixed:
- sed -i '' syntax (blocks GNU sed on Linux)
- /opt/homebrew hardcoded paths (don't exist on Linux)
- Added systemd user service for Linux auto-start
- Updated outdated Linux warnings in VERIFY.md
Platform improvements:
- USERNAME fallback for edge cases (${USER:-$(whoami)})
- Conditional Homebrew PATH detection
- systemd best practices (%h, WorkingDirectory, linger)
- Cross-platform documentation in new PLATFORM.md
Testing:
All fixes designed for Ubuntu 24.04 (WSL2) with systemd 255.
No macOS functionality changed - pure additions for Linux.
Addresses community feedback about Linux support gaps.
🤖 Generated with [Claude Code](https://claude.com/claude-code)
Co-Authored-By: Claude Sonnet 4.5 <noreply@anthropic.com>
Contributor
Author
Testing Complete ✅All 4 fixes have been tested on Ubuntu 24.04 (WSL2) and work correctly:
Test environment:
Complete test results documented in No issues found - ready for review. |
Owner
|
Merged! Thanks for the continued contributions - the Google TTS work and now this Linux compatibility work are both solid additions. The PLATFORM.md roadmap is a nice touch for future contributors. 🙏 |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Summary
Fixes 4 critical Linux compatibility issues, enabling PAI installation on Google Gemini CLI and other Linux environments.
Motivation
After contributing Google Cloud TTS (#285), we tested PAI installation on Linux (Ubuntu/WSL2) to enable use with Google Gemini CLI. Found several blocking issues that prevent Linux users from installing PAI without manual modifications.
What's Changed
Key Improvements
sed Command: Added USERNAME fallback and platform detection for macOS BSD vs GNU sed
PATH Detection: Conditional Homebrew path only if directory exists
systemd Auto-Start: Full Linux support using systemd best practices (percent-h, WorkingDirectory, loginctl linger)
Documentation: New PLATFORM.md tracking all 22 platform issues with community contribution roadmap
Testing
Tested on Ubuntu 24.04 (WSL2) with systemd 255:
Breaking Changes
None. macOS functionality unchanged. Pure additions for Linux support.
Aligns with PAI Principles
Simple platform detection via uname, UNIX philosophy with native tools, transparent user-visible logic, deterministic behavior, removes macOS-only assumptions.
Generated with Claude Code