[Bug 1554050] Review Request: gli - OpenGL Image (GLI)

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



https://bugzilla.redhat.com/show_bug.cgi?id=1554050



--- Comment #2 from Ian Hattendorf <ian@xxxxxxxxxxxxxxxxx> ---
(In reply to Robert-André Mauchin from comment #1)
>  - This shouldn't be needed, instead make your devel subpackage noarch.
> 
> # Header only library
> %global debug_package %{nil}
> 
>  - Not needed in Fedora:
> 
> Group:		Development/Libraries
> 
>  - Test failures on bid-endian machines
> 
> Have you reported it upstream?
> 
> 
> I'm not able to sponsor you, try introducing yourself on the fedora-devel
> mailing list and do informal reviews to show you understand the guidelines.

Thanks for the comment. I've removed the deprecated Group tag and added a link
to the upstream issue report. I've also added BuildRequires: gcc-c++.

Regarding noarch, I thought that wasn't recommended for header only libraries?

"It may be tempting to make the header library package noarch, since the header
files themselves are simply text. However, a library should have tests which
should be run on all architectures. Also, the install process may modify the
installed headers depending on the build architecture. For these reasons,
header-only packages must not be marked noarch."
https://fedoraproject.org/wiki/Packaging:Guidelines#Do_not_use_noarch

If that's outdated or not recommended in this case, it should be easy enough to
switch to noarch. The cmake files would need to be moved from _libdir to
_datadir, and the gliVersion.cmake file patched to ignore system pointer size.

Spec URL:
https://copr-be.cloud.fedoraproject.org/results/ianhattendorf/gli/fedora-rawhide-x86_64/00727593-gli/gli.spec
SRPM URL:
https://copr-be.cloud.fedoraproject.org/results/ianhattendorf/gli/fedora-rawhide-x86_64/00727593-gli/gli-0.8.2.0-1.fc29.src.rpm

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
_______________________________________________
package-review mailing list -- package-review@xxxxxxxxxxxxxxxxxxxxxxx
To unsubscribe send an email to package-review-leave@xxxxxxxxxxxxxxxxxxxxxxx




[Index of Archives]     [Fedora Users]     [Fedora Desktop]     [Fedora SELinux]     [Yosemite Conditions]     [KDE Users]

  Powered by Linux