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



--- Comment #28 from Denis Fateyev <[email protected]> ---
Looked at the current version in rawhide. Just a small nit-pick:

1) Please use "install -Dpm 0644 doc/man/man3/msgpuck.h.3*
%{buildroot}%{_mandir}/man3/" instead of "cp -f" to preserve files timestamps;

2) Though I'm not really insisting, but pointing out full BRs is according the
current guidelines. As pointed above, you should also add `make` and
`coreutils`;

3) Changelog list will shortly become messy, e.g. during the next mass rebuild
by rel-eng: they'll add a changelog entry with an empty line in the end making
the whole list ragged. As also pointed above, it would better to use an empty
line as the changelog items delimiter.

-- 
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
[email protected]
https://admin.fedoraproject.org/mailman/listinfo/package-review

Reply via email to