fix(release): handle missing dnf in aarch64 manylinux container#53
fix(release): handle missing dnf in aarch64 manylinux container#53
Conversation
The aarch64 manylinux_2_28 container does not expose dnf in PATH, causing before-script to exit 127. Fall through dnf→yum→apt-get so protoc installs regardless of which package manager is available. Closes #52
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: Organization UI Review profile: ASSERTIVE Plan: Pro Run ID: 📒 Files selected for processing (1)
📝 WalkthroughSummary by CodeRabbit
WalkthroughThe PR modifies the embedded wheel build step's Linux setup script to conditionally install ChangesCI Build Script Fix
Estimated code review effort🎯 1 (Trivial) | ⏱️ ~3 minutes 🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing Touches🧪 Generate unit tests (beta)
Review rate limit: 4/5 reviews remaining, refill in 12 minutes. Comment |
|



Summary
Build embedded / aarch64 (ubuntu-latest)fails in Release v1.0.1 withdnf: command not found→ docker exits 127dnfin PATH; before-script falls through with exit code 127dnf→yum→apt-getso protoc installs regardless of package managerCloses #52