Bug 1811647

Summary: %{?_isa} qualifier unnecessary / broken for BuildRequires
Product: [Fedora] Fedora Reporter: Fabio Valentini <decathorpe>
Component: CGALAssignee: Laurent Rineau <laurent.rineau__fedora>
Status: CLOSED CURRENTRELEASE QA Contact: Fedora Extras Quality Assurance <extras-qa>
Severity: unspecified Docs Contact:
Priority: unspecified    
Version: rawhideCC: laurent.rineau__fedora
Target Milestone: ---   
Target Release: ---   
Hardware: Unspecified   
OS: Unspecified   
Whiteboard:
Fixed In Version: Doc Type: If docs needed, set a value
Doc Text:
Story Points: ---
Clone Of: Environment:
Last Closed: 2020-03-09 15:18:45 UTC Type: Bug
Regression: --- Mount Type: ---
Documentation: --- CRM:
Verified Versions: Category: ---
oVirt Team: --- RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: --- Target Upstream Version:
Embargoed:

Description Fabio Valentini 2020-03-09 13:03:42 UTC
These lines in the CGAL .spec file are causing issues:

BuildRequires: qt5-qtbase-devel%{?_isa} >= %{qt_version}
BuildRequires: qt5-qtsvg-devel%{?_isa} >= %{qt_version}
BuildRequires: qt5-qtscript-devel%{?_isa} >= %{qt_version}
BuildRequires: qt5-qttools-devel%{?_isa} >= %{qt_version}

See: https://src.fedoraproject.org/rpms/CGAL/blob/master/f/CGAL.spec#_30-33

They cause broken dependencies on every architecture that the source package was *not* built on. For example, the currently available source package from rawhide requires:

qt5-qtbase-devel(armv7hl-32) >= 5.9
qt5-qtscript-devel(armv7hl-32) >= 5.9
qt5-qtsvg-devel(armv7hl-32) >= 5.9
qt5-qttools-devel(armv7hl-32) >= 5.9

Which creates broken dependencies on x86_64 (and every arch other than armv7hl).

Please drop the %{?_isa} qualifier from these BuildRequires, they are unnecessary and create broken dependencies.

Comment 1 Laurent Rineau 2020-03-09 15:18:45 UTC
Thanks for the report.

That is fixed by https://bodhi.fedoraproject.org/updates/FEDORA-2020-535ea4f40c.

Do you think I should backport the fix to F30/F31/F32, or just wait for the next opportunity to rebuild the package for those releases?

Comment 2 Fabio Valentini 2020-03-09 15:34:41 UTC
Thanks for fixing it!

Waiting for the next update to push this for older releases works for me. Thanks!