https://bugzilla.redhat.com/show_bug.cgi?id=1565848
--- Comment #24 from Till Hofmann thofmann@fedoraproject.org --- Some remarks:
- The license should be GPLv3+, not GPLv3.
- As Robert-André pointed out, you don't need to Require python.
- The shebang replacement should keep the timestamp. Your version is also not safe, as it would substitute "/usr/bin/env python" anywhere in the file, and it would replace "/usr/bin/env python3" by "/usr/bin/python33". I usually use the following snippet in %install instead: for f in %{buildroot}/%{_bindir}/* ; do sed -i.orig "s:^#!/usr/bin/env\s+python\s?$:#!%{__python3}:" $f touch -r $f.orig $f rm $f.orig done
Note that if the shebang is "/usr/bin/env python3", then that is fine, because the mangler will automatically change it to "/usr/bin/python3", see [1].
- In the future, please follow the pattern with URLs to the SPEC and SRPM on the first two lines, as in the original post. The URLs should point to a raw SPEC and a directly downloadable SRPM. This allows a reviewer to use fedora-review.
- The file section can be simplified, e.g., instead of %dir %{_libdir}/bear/ %{_libdir}/bear/libear.so you can simply write %{_libdir}/bear Similarly for the docs.
- You list some doc files twice. If you list a file such as "%doc README.md", then you don't need to list it again. The build system already installs those files, so either you list them with %doc %{_docdir}/bear, or you remove them in %install and install them with %doc README.md etc. You also have two copies of COPYING. No need to have it in the doc dir, although that's not a big issue either.
- Why do you have conditionals on %{?fedora}?
[1] https://fedoraproject.org/wiki/Packaging:Guidelines#Shebang_lines