fix: rust-nalgebra-macros0.2: pin to Fedora 43 build commit 0e40a33cf88e#16388
Conversation
Pin rust-nalgebra-macros0.2 to upstream commit 0e40a33cf88e651400bc10591e1a50f5daa97d10 (Fedora dist-git: rust-nalgebra-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-nalgebra-macros0.2 component to a specific Fedora 43 dist-git commit so azldev fetches sources/spec content matching the commit used for the current Fedora 43 build.
Changes:
- Add a dedicated component definition file for
rust-nalgebra-macros0.2that pinsspec.upstream-committo0e40a33cf88e651400bc10591e1a50f5daa97d10. - Remove the now-redundant empty entry for
rust-nalgebra-macros0.2fromcomponents-full.toml.
Reviewed changes
Copilot reviewed 2 out of 2 changed files in this pull request and generated 1 comment.
| File | Description |
|---|---|
| base/comps/rust-nalgebra-macros0.2/rust-nalgebra-macros0.2.comp.toml | Introduces an upstream-commit pin for Fedora 43 to align fetched sources with the Fedora build commit. |
| base/comps/components-full.toml | Removes the component stub entry since the component now has a dedicated *.comp.toml. |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
| [components.'rust-nalgebra-macros0.2'] | ||
| # 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.
The pin comment is missing a concrete reference to the Fedora dist-git commit (and ideally the Fedora build/NVR) that this override is intended to match. Please add a link to the src.fedoraproject.org commit URL (with the f43 branch) so future readers can verify when it’s safe to drop the pin and what it corresponds to.
| # in distro/azurelinux.distro.toml advances past this commit. | |
| # in distro/azurelinux.distro.toml advances past this commit. | |
| # Fedora dist-git reference: https://src.fedoraproject.org/rpms/rust-nalgebra-macros/c/0e40a33cf88e651400bc10591e1a50f5daa97d10?branch=f43 |
Pin rust-nalgebra-macros0.2 to upstream commit 0e40a33cf88e651400bc10591e1a50f5daa97d10 (Fedora dist-git: rust-nalgebra-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