Skip to content

spec: Use ExclusiveArch for supported architectures#221

Merged
cgwalters merged 1 commit intomainfrom
add-packit-arch-patch
Mar 10, 2026
Merged

spec: Use ExclusiveArch for supported architectures#221
cgwalters merged 1 commit intomainfrom
add-packit-arch-patch

Conversation

@gursewak1997
Copy link
Collaborator

@gursewak1997 gursewak1997 commented Mar 9, 2026

Only build RPM packages for x86_64 and aarch64, which are the architectures with full support and testing. This prevents build failures on ppc64le/s390x where the unit tests fail. This doesn't block developers from building with cargo on any architecture for local experimentation.

Copy link

@gemini-code-assist gemini-code-assist bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Code Review

This pull request updates the packit configuration to restrict Fedora builds to supported architectures (x86_64, aarch64) by using ExclusiveArch. No security vulnerabilities were found. However, a potential issue with the verification step was identified that could lead to incorrect builds if the substitution fails. Suggestions have been provided to make the verification more robust and to simplify commands by removing redundant bash -c wrappers.

cgwalters
cgwalters previously approved these changes Mar 9, 2026
@gursewak1997 gursewak1997 force-pushed the add-packit-arch-patch branch 2 times, most recently from 93f23d9 to 899bdef Compare March 9, 2026 19:50
Only build RPM packages for x86_64 and aarch64, which are the
architectures with full support and testing. This prevents build
failures on ppc64le/s390x where the unit tests fail.
This doesn't block developers from building with cargo on any
architecture for local experimentation.

Signed-off-by: gursewak1997 <gursmangat@gmail.com>
@gursewak1997 gursewak1997 force-pushed the add-packit-arch-patch branch from 899bdef to 7bacc1c Compare March 9, 2026 21:44
@gursewak1997 gursewak1997 changed the title packit: Use ExclusiveArch for Fedora builds spec: Use ExclusiveArch for supported architectures Mar 9, 2026
@cgwalters cgwalters enabled auto-merge (rebase) March 9, 2026 21:58
@cgwalters cgwalters merged commit f03154e into main Mar 10, 2026
26 of 30 checks passed
@gursewak1997 gursewak1997 deleted the add-packit-arch-patch branch March 10, 2026 19:00
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants