fix: rust-strum0.27: pin to Fedora 43 build commit 66d4daf5de82#16394
fix: rust-strum0.27: pin to Fedora 43 build commit 66d4daf5de82#16394christopherco merged 1 commit intotomls/base/mainfrom
Conversation
Pin rust-strum0.27 to upstream commit 66d4daf5de82de09f4b565dc35a389bc631eada4 (Fedora dist-git: rust-strum) to fetch sources from the commit used to build the current f43 package.
There was a problem hiding this comment.
Pull request overview
Pins the rust-strum0.27 component to a specific Fedora 43 dist-git commit so Azure Linux can fetch the same sources/spec used for the currently built Fedora 43 package.
Changes:
- Add a dedicated
rust-strum0.27.comp.tomlthat pins the upstream spec to Fedora 43 commit66d4daf5de82de09f4b565dc35a389bc631eada4. - Remove the now-redundant
rust-strum0.27entry fromcomponents-full.toml(since it has a dedicated component file).
Reviewed changes
Copilot reviewed 2 out of 2 changed files in this pull request and generated 1 comment.
| File | Description |
|---|---|
| base/comps/rust-strum0.27/rust-strum0.27.comp.toml | Introduces an upstream-commit pin for rust-strum0.27 against Fedora 43. |
| base/comps/components-full.toml | Drops the rust-strum0.27 stub entry to avoid duplicate/empty component definitions. |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
| [components.'rust-strum0.27'] | ||
| # Pin to the f43 commit used to build the current Fedora 43 package. | ||
| # TODO: Drop this override once the default Fedora 43 snapshot | ||
| # in distro/azurelinux.distro.toml advances past this commit. |
There was a problem hiding this comment.
Pin comment is missing a reference URL to the exact Fedora dist-git commit being pinned (the repo convention is to include full external links when pinning to an upstream commit for traceability). Please add a line like # Fedora commit: https://src.fedoraproject.org/rpms/rust-strum/c/66d4daf5de82de09f4b565dc35a389bc631eada4?branch=f43 near the TODO/pin rationale so future readers can verify the source of the pin.
| # in distro/azurelinux.distro.toml advances past this commit. | |
| # in distro/azurelinux.distro.toml advances past this commit. | |
| # Fedora commit: https://src.fedoraproject.org/rpms/rust-strum/c/66d4daf5de82de09f4b565dc35a389bc631eada4?branch=f43 |
Pin rust-strum0.27 to upstream commit 66d4daf5de82de09f4b565dc35a389bc631eada4 (Fedora dist-git: rust-strum) to fetch sources from the commit used to build the current f43 package.
Merge Checklist
All boxes should be checked before merging the PR (just tick any boxes which don't apply to this PR)
*-staticsubpackages, etc.) have had theirReleasetag incremented../cgmanifest.json,./toolkit/scripts/toolchain/cgmanifest.json,.github/workflows/cgmanifest.json)./LICENSES-AND-NOTICES/SPECS/data/licenses.json,./LICENSES-AND-NOTICES/SPECS/LICENSES-MAP.md,./LICENSES-AND-NOTICES/SPECS/LICENSE-EXCEPTIONS.PHOTON)*.signatures.jsonfilessudo make go-tidy-allandsudo make go-test-coveragepassSummary
What does the PR accomplish, why was it needed?
Change Log
Does this affect the toolchain?
YES/NO
Associated issues
Links to CVEs
Test Methodology