https://bugzilla.redhat.com/show_bug.cgi?id=1321455
Jan Včelák jv+fedora@fcelda.cz changed:
What |Removed |Added ---------------------------------------------------------------------------- Flags|needinfo?(jv+fedora@fcelda. |needinfo?(thozza@redhat.com |cz) |)
--- Comment #2 from Jan Včelák jv+fedora@fcelda.cz --- Thank you. Updated.
Spec URL: https://jvcelak.fedorapeople.org/review/knot-resolver/1.0.0-0.2.79a8440/knot... SRPM URL: https://jvcelak.fedorapeople.org/review/knot-resolver/1.0.0-0.2.79a8440/knot...
(In reply to Tomas Hozza from comment #1)
- Package does not contain duplicates in %files. Note: warning: File listed twice: /var/lib/kresd/root.keys See: http://fedoraproject.org/wiki/Packaging/Guidelines#DuplicateFiles
Fixed.
- Packages should try to preserve timestamps of original installed files.
Fixed.
- Files outside of /usr SHOULD be owned by root:root (https://fedoraproject.org/wiki/Packaging:Guidelines#File_Permissions)
I've changed UID to root where possible. And kept GID to kresd where necessary. Files inside /var/lib are state information and have to be writable by the daemon.
- Fully versioned dependency in subpackages if applicable. Note: No Requires: %{name}%{?_isa} = %{version}-%{release} in knot- resolver-devel
Fixed.
- Once you resolve issues with building the documentation, please package is as well, because currently there is no man page for the daemon.
The documentation building can't be fixed easily. The RTD theme cannot be loaded by Python 2 Sphinx on Fedora, and Breathe is incompatible with Python 3.
I've updated the package to current git master because some of the building issues were resolved. The manual page is now installed. And unit tests results are temporarily ignored (one failure is present and should be fixed in the gold release).