Skip to content
This repository was archived by the owner on Oct 30, 2026. It is now read-only.
This repository was archived by the owner on Oct 30, 2026. It is now read-only.

fix(release): use if/else for protoc install, add apt-get update for aarch64 #55

Closed

Description

@polaz

Problem

Build embedded / aarch64 (ubuntu-latest) still fails in v1.0.2:

  • aarch64 uses ghcr.io/rust-cross/manylinux_2_28-cross:aarch64 (Debian-based) → needs apt-get update before install
  • The || chain in before-script also causes both dnf AND yum to run on x86_64 (yum=alias for dnf on AlmaLinux 8), then apt-get is tried and fails with 'command not found'

Fix

Replace || chain with if/else: check for dnf first (manylinux x86_64 = AlmaLinux 8), fall back to apt-get update && apt-get install (rust-cross aarch64 = Debian).

Activity

  1. coderabbitai commented on May 3, 2026

    @coderabbitai
    ⚠️ Possible Duplicate Issue(s)
    🔗 Related PRs

    #53 - fix(release): handle missing dnf in aarch64 manylinux container [closed]


    📝 Issue Planner

    Check the box below or use the @coderabbitai plan command to generate an implementation plan and prompts that you can use with your favorite coding assistant.

    • Create Plan

    🧪 Issue enrichment is currently in open beta.

    You can configure auto-planning by selecting labels in the issue_enrichment configuration.

    To disable automatic issue enrichment, add the following to your .coderabbit.yaml:

    issue_enrichment:
      auto_enrich:
        enabled: false

    💬 Have feedback or questions? Drop into our discord!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

No one assigned

    Labels

    No labels
    No labels

    Type

    No type

    Projects

    No projects

      Milestone

      No milestone

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions