[Bug 1295217] Review Request: msgpuck - a MsgPack serialization library in a self-contained header file

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

 



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



--- Comment #28 from Denis Fateyev <denis@xxxxxxxxxxx> ---
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
package-review@xxxxxxxxxxxxxxxxxxxxxxx
https://admin.fedoraproject.org/mailman/listinfo/package-review




[Index of Archives]     [Fedora Legacy]     [Fedora Desktop]     [Fedora SELinux]     [Yosemite News]     [KDE Users]     [Fedora Tools]