feat(rust-cpubits): add rust-cpubits component - #18446
Draft
Autumn Nash (WithEnoughCoffee) wants to merge 2 commits into
Draft
feat(rust-cpubits): add rust-cpubits component#18446Autumn Nash (WithEnoughCoffee) wants to merge 2 commits into
Autumn Nash (WithEnoughCoffee) wants to merge 2 commits into
Conversation
Adds rust-cpubits, a new leaf crate not previously packaged in AZL. It was first packaged in Fedora after AZL's distro-default snapshot date (2026-02-24), so no commit exists at that snapshot and an explicit upstream-commit pin (Fedora f43 HEAD) is required. Split out of PR 5/9 (#18405, zip transitive prereqs) so it can be built and tagged into the azl4-build repo ahead of that PR: koji scratch builds only resolve dynamic BuildRequires against packages already tagged in the target repo, and rust-aes 0.9.1 (added in #18405) requires crate(cpubits/default) >= 0.1.0, < 0.2.0. Building this leaf package first unblocks scratch-testing rust-aes. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com> Copilot-Session: 8e36bd6e-7367-4e7c-918b-4ccf3155a55a
Copilot started reviewing on behalf of
Autumn Nash (WithEnoughCoffee)
August 14, 2026 21:13
View session
Contributor
There was a problem hiding this comment.
Pull request overview
Adds the rust-cpubits leaf crate required by the upcoming Rust dependency alignment.
Changes:
- Pins Fedora 43’s
rust-cpubits0.1.1 component. - Adds its lock file, rendered RPM spec, and source checksum.
- Keeps
%checkenabled.
Reviewed changes
Copilot reviewed 3 out of 4 changed files in this pull request and generated 1 comment.
| File | Description |
|---|---|
base/comps/rust-cpubits/rust-cpubits.comp.toml |
Defines and documents the pinned component. |
locks/rust-cpubits.lock |
Locks upstream resolution and inputs. |
specs/r/rust-cpubits/rust-cpubits.spec |
Provides the rendered RPM packaging. |
specs/r/rust-cpubits/sources |
Records the crate checksum. |
💡 Configure MCP servers for context-aware, tailored reviews. Learn more in the docs.
| # HEAD) is required. Needed transitively by rust-aes 0.9.1 | ||
| # (crate(cpubits/default) >= 0.1.0, < 0.2.0) as part of the uv | ||
| # crate-alignment cascade. | ||
| spec = { type = "upstream", upstream-distro = { name = "fedora", version = "43" }, upstream-commit = "4e784900139bc3b9b30c59a0e892319834cca2bb" } |
Contributor
Author
|
/azp run |
|
Azure Pipelines: 2 pipeline(s) were filtered out due to trigger conditions. |
Per review feedback, drop PR-context details (dependency chain justification, uv crate-alignment cascade narrative) that won't be relevant once merged; keep only the brief justification for the upstream-commit pin.
Copilot started reviewing on behalf of
Autumn Nash (WithEnoughCoffee)
August 14, 2026 23:42
View session
Contributor
There was a problem hiding this comment.
Pull request overview
Copilot reviewed 3 out of 4 changed files in this pull request and generated no new comments.
Suppressed comments (1)
base/comps/rust-cpubits/rust-cpubits.comp.toml:1
- The stated validation stops at a successful build and
%check, but repository policy requires every new component to be inspected, installed, and smoke-tested in a mock chroot. Please add those results before merge, or explicitly document why this source-only crate cannot be smoke-tested and what equivalent validation was performed.
[components.rust-cpubits]
Contributor
Author
|
/azp run |
|
Azure Pipelines: 2 pipeline(s) were filtered out due to trigger conditions. |
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
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
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
Adds
rust-cpubits, a new leaf crate not previously packaged in AZL.Split out ahead of #18405 (zip transitive prereqs, PR 5/9): koji scratch builds only resolve dynamic BuildRequires against packages already tagged in the target repo (
azl4-build), andrust-aes0.9.1 (added in #18405) requirescrate(cpubits/default) >= 0.1.0, < 0.2.0. Landing this leaf package here first, via the official (non-scratch) post-merge build, gets it tagged for real so #18405's scratch-build check can resolve it.It was first packaged in Fedora after AZL's distro-default snapshot date (2026-02-24), so no commit exists at that snapshot and an explicit
upstream-commitpin (Fedora f43 HEAD) is used.Testing
azldev component update -a --check-only -q— clean, no render/lock drift.azldev comp build -p rust-cpubits— local mock-chroot build succeeds; producesrust-cpubits+default-develandrust-cpubits-develnoarch RPMs;%checkruns cleanly.Related
rust-cmov), also a prerequisite for feat(zip-prereqs): bump aes/const-oid/digest/hmac/md-5/pbkdf2/sha1/sha2/streebog + compat packages, proptest #18405.