fix: rust-strum_macros0.27: pin to Fedora 43 build commit 3ffef61b4a61#16387
Conversation
Pin rust-strum_macros0.27 to upstream commit 3ffef61b4a61d588d3fa2ade20da7ff758590d54 (Fedora dist-git: rust-strum_macros) 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-strum_macros0.27 component to a specific Fedora 43 dist-git commit so the spec/source resolution matches the commit used for the current Fedora 43 build, improving reproducibility of the imported packaging.
Changes:
- Add a dedicated
rust-strum_macros0.27.comp.tomlthat pins the upstream spec to commit3ffef61b4a61d588d3fa2ade20da7ff758590d54from Fedora 43. - Remove the now-dedicated
rust-strum_macros0.27entry fromcomponents-full.tomlto avoid duplicate/empty definitions.
Reviewed changes
Copilot reviewed 2 out of 2 changed files in this pull request and generated 1 comment.
| File | Description |
|---|---|
| base/comps/rust-strum_macros0.27/rust-strum_macros0.27.comp.toml | Introduces the Fedora 43 upstream-commit pin for rust-strum_macros0.27. |
| base/comps/components-full.toml | Removes the component entry now that the component has a dedicated .comp.toml file. |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
| [components.'rust-strum_macros0.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.
Add a reference link to the pinned Fedora dist-git commit (and optionally the Koji build/NVR) in the comment block. Other commit pins in base/comps (e.g., golang) include a direct src.fedoraproject.org commit URL, which makes it much easier to audit why this hash was chosen and when it can be dropped.
| # in distro/azurelinux.distro.toml advances past this commit. | |
| # in distro/azurelinux.distro.toml advances past this commit. | |
| # Fedora dist-git commit: https://src.fedoraproject.org/rpms/rust-strum_macros/c/3ffef61b4a61d588d3fa2ade20da7ff758590d54 |
Pin rust-strum_macros0.27 to upstream commit 3ffef61b4a61d588d3fa2ade20da7ff758590d54 (Fedora dist-git: rust-strum_macros) 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