https://bugzilla.redhat.com/show_bug.cgi?id=1907238
Zbigniew Jędrzejewski-Szmek zbyszek@in.waw.pl changed:
What |Removed |Added ---------------------------------------------------------------------------- Status|NEW |POST Assignee|nobody@fedoraproject.org |zbyszek@in.waw.pl Flags| |fedora-review+
--- Comment #4 from Zbigniew Jędrzejewski-Szmek zbyszek@in.waw.pl ---
BuildRequires: cmake gcc gcc-c++
One-per-line is preferred nowadays.
%{_bindir}/*
Maybe '%{_bindir}/exr*' ? This will make it less likely to something unintended to slip in on upgrades.
rpmlint: openexr.src: W: spelling-error %description -l en_US containes -> contained, contains, containers
# Is it OK to dump the libraries in site-packages?
It means that the modules 'iex' and 'imath' will be importable in the global namespace. It's certainly allowed in general. I think the name is a bit generic in this case, but it's not something we have influence over.
+ license is acceptable for Fedora (BSD 3 clause) + license is specified correctly + builds and installs OK + BR and Requires look correct + Provides and Obsoletes look correct took
Package is APPROVED.