breadlock ci.yml tries to pacman-install a nonexistent clippy package #8

Closed
opened 2026-09-11 22:04:44 +08:00 by Breadway · 1 comment
Owner

The "Install build deps" step in .forgejo/workflows/ci.yml pacman-installs a package literally named clippy, which does not exist as a separate Arch package (clippy ships as a rustup component or bundled differently, not a pacman target). Every push to main and every PR fails ci.yml's check job instantly with "error: target not found: clippy", before rustfmt/clippy/test/build ever run.

Found while promoting breadlock to v0.3.0 for BOS 0.6.2. Does not block releases: package.yml (the actual release build) has its own separate deps list that does not include "clippy" and builds fine. This only breaks the PR/branch CI gate, which has presumably been red for a while.

Fix: compare against how bread-ecosystem or another repo's ci.yml installs clippy (likely rustup component add clippy after installing rustup, or just drop the pacman clippy target and use cargo clippy from whatever toolchain rust/cargo already provides if it's bundled).

The "Install build deps" step in .forgejo/workflows/ci.yml pacman-installs a package literally named `clippy`, which does not exist as a separate Arch package (clippy ships as a rustup component or bundled differently, not a pacman target). Every push to main and every PR fails ci.yml's check job instantly with "error: target not found: clippy", before rustfmt/clippy/test/build ever run. Found while promoting breadlock to v0.3.0 for BOS 0.6.2. Does not block releases: package.yml (the actual release build) has its own separate deps list that does not include "clippy" and builds fine. This only breaks the PR/branch CI gate, which has presumably been red for a while. Fix: compare against how bread-ecosystem or another repo's ci.yml installs clippy (likely `rustup component add clippy` after installing rustup, or just drop the pacman clippy target and use `cargo clippy` from whatever toolchain rust/cargo already provides if it's bundled).
Author
Owner

This fix is already present on origin/main in commit 8a7a570162. It removes clippy and rustfmt from the pacman install list, since Arch's rust package provides both. The current main commit 05574a0 has a successful CI push check (run 28, 4m53s), confirming the workflow now passes. No new branch or PR is needed.

This fix is already present on origin/main in commit 8a7a5701620bd98c2c1db243aabb097f2d063d61. It removes clippy and rustfmt from the pacman install list, since Arch's rust package provides both. The current main commit 05574a0 has a successful CI push check (run 28, 4m53s), confirming the workflow now passes. No new branch or PR is needed.
Sign in to join this conversation.
No labels
No milestone
No project
No assignees
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference
Breadway/breadlock#8
No description provided.