packaging(arch): add openhuman-bin AUR recipe#2428
Conversation
📝 WalkthroughWalkthroughAdds an Arch AUR recipe ( ChangesArch Linux AUR Distribution
Estimated code review effort🎯 2 (Simple) | ⏱️ ~10 minutes Poem
🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. 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. Comment |
b6941be to
262288a
Compare
graycyrus
left a comment
There was a problem hiding this comment.
Nice work — this is a well-structured AUR recipe that directly addresses #2417 (Interpreter not found!) and advances #1593. The AppImage extraction approach, SHARUN_LDNAME default, and bundled LD_LIBRARY_PATH are all solid choices for Arch-family distros.
A couple of minor suggestions below, nothing blocking.
| package() { | ||
| install -d "${pkgdir}/opt/openhuman" | ||
| cp -a --no-preserve=ownership "${srcdir}/squashfs-root/." \ | ||
| "${pkgdir}/opt/openhuman/" |
There was a problem hiding this comment.
[minor] cp -a copies the entire squashfs-root into /opt/openhuman/, including unused top-level files from the AppImage (AppRun, root .desktop, root icon, etc.). Only shared/ is actually referenced by the launcher.
Consider installing just the subtree the launcher needs:
cp -a --no-preserve=ownership "${srcdir}/squashfs-root/shared" \
"${pkgdir}/opt/openhuman/shared"This trims the install footprint and avoids shipping dead files.
|
|
||
| package() { | ||
| install -d "${pkgdir}/opt/openhuman" | ||
| cp -a --no-preserve=ownership "${srcdir}/squashfs-root/." \ |
There was a problem hiding this comment.
[minor] cp -a preserves the file modes from inside the AppImage (-a implies --preserve=mode). Some AppImages pack files with overly permissive modes (e.g. 0777). Worth normalizing after the copy:
chmod -R u=rwX,go=rX "${pkgdir}/opt/openhuman"
Summary
openhuman-binpackage recipe underpackages/arch/openhuman-bin.makepkg, and launches the innershared/bin/OpenHumanbinary with the bundled library path./usr/bin/openhumanlauncher,.SRCINFO, and release-bump notes.Submission Checklist
Validation
git diff --cached --checkOpenHuman_0.54.0_amd64.AppImage.openhuman,openhuman.desktop, andopenhuman.svgSHA-256 checksums matchPKGBUILDand.SRCINFO.PKGBUILDand.SRCINFOfields.Validation Blocked
makepkg --syncdeps --clean --cleanbuild --force- blocked: current host is Windows, not Arch Linux.bash -n packages/arch/openhuman-bin/PKGBUILD- blocked:bash.exeresolves to WSL, and WSL2 is not enabled on this host (HCS_E_HYPERV_NOT_INSTALLED).Related
AI Authored PR Metadata (required for Codex/Linear PRs)
Linear Issue
Commit & Branch
codex/1593-arch-aur-package262288aa607b1599749984e8701e9fa999ec3267Validation Run
Behavior Changes
Parity Contract
.deb, Homebrew, and Windows installer paths remain unchanged.openhuman-binpackages the same official x86_64 AppImage release asset, but extracts it during package build so Arch-family users do not launch the AppImage runtime directly.Duplicate / Superseded PR Handling
Summary by CodeRabbit
New Features
Documentation